Skip to content

🚧 Add space branding to invite page#2314

Merged
lukevella merged 1 commit intomainfrom
luke/ral-1126-space-branding-invite-page
Apr 5, 2026
Merged

🚧 Add space branding to invite page#2314
lukevella merged 1 commit intomainfrom
luke/ral-1126-space-branding-invite-page

Conversation

@lukevella
Copy link
Copy Markdown
Owner

@lukevella lukevella commented Apr 5, 2026

Summary by CodeRabbit

  • New Features
    • Poll branding with custom primary colors now displays on admin and invite pages when enabled in space settings.

♻️ Only apply poll branding when primary color is set
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app Ready Ready Preview, Comment Apr 5, 2026 0:43am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
landing Skipped Skipped Apr 5, 2026 0:43am

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 5, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 5be16b04-bdaf-48b2-b73f-d3de80d8e4fe

📥 Commits

Reviewing files that changed from the base of the PR and between e63d638 and a85d0e8.

📒 Files selected for processing (3)
  • apps/web/src/app/[locale]/(optional-space)/poll/[urlId]/admin-page.tsx
  • apps/web/src/app/[locale]/invite/[urlId]/invite-page.tsx
  • apps/web/src/components/poll/poll-branding.tsx

Walkthrough

A new PollBranding component is added that applies custom primary color theming to polls via CSS variables. The component is conditionally rendered on admin and invite pages when branding is enabled and a primary color is configured.

Changes

Cohort / File(s) Summary
Poll Branding Integration
apps/web/src/app/[locale]/.../(optional-space)/poll/[urlId]/admin-page.tsx, apps/web/src/app/[locale]/invite/[urlId]/invite-page.tsx
Added conditional rendering of PollBranding component on both pages when poll.space?.showBranding is true and poll.space.primaryColor exists.
Poll Branding Component
apps/web/src/components/poll/poll-branding.tsx
New client-side component that accepts primaryColor prop, derives CSS variable values via getPrimaryColorVars(), and renders inline <style> tag defining --primary and --primary-foreground variables for light/dark theme scopes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 A splash of color hops onto the page,
Where polls now wear their brand with grace,
CSS variables dance and play,
Making every poll display its own way! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding space branding functionality to the invite page through a new PollBranding component.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch luke/ral-1126-space-branding-invite-page

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@lukevella lukevella merged commit 630e585 into main Apr 5, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant