Skip to content

Have the paywall support all themes#1072

Merged
saddlepaddle merged 2 commits intomainfrom
paywall-light
Jan 30, 2026
Merged

Have the paywall support all themes#1072
saddlepaddle merged 2 commits intomainfrom
paywall-light

Conversation

@saddlepaddle
Copy link
Copy Markdown
Collaborator

@saddlepaddle saddlepaddle commented Jan 30, 2026

Description

Related Issues

Type of Change

  • Bug fix
  • New feature
  • Documentation
  • Refactor
  • Other (please describe):

Testing

Screenshots (if applicable)

Additional Notes

Summary by CodeRabbit

  • Style
    • Updated the visual theme across paywall feature preview components to use a new design token system. Colors and styling for backgrounds, borders, text, icons, and status indicators have been refreshed throughout demo displays including cloud workspaces, integrations, mobile app, tasks, and team collaboration sections. No functional changes.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 30, 2026

Warning

Rate limit exceeded

@saddlepaddle has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 19 minutes and 24 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📝 Walkthrough

Walkthrough

This PR updates styling tokens across six paywall demo components, replacing dark theme color classes and hard-coded values with semantic design tokens (bg-card, bg-muted, text-muted-foreground, foreground, etc.) to implement consistent theming throughout the feature preview UI.

Changes

Cohort / File(s) Summary
Demo Components - Styling Token Updates
apps/desktop/src/renderer/components/Paywall/components/FeaturePreview/components/CloudWorkspacesDemo/CloudWorkspacesDemo.tsx, IntegrationsDemo/IntegrationsDemo.tsx, MobileAppDemo/MobileAppDemo.tsx, TasksDemo/TasksDemo.tsx, TeamCollaborationDemo/TeamCollaborationDemo.tsx
Replaced dark theme color tokens (white/black variants with opacity) with semantic design tokens across containers, headers, labels, icons, and status indicators. Updates include background (bg-card, bg-muted), text (text-muted-foreground, text-foreground), and border colors (border-foreground, border-card).
Sidebar Component
apps/desktop/src/renderer/components/Paywall/components/FeatureSidebar/FeatureSidebar.tsx
Updated root container background from bg-neutral-900 to bg-card for theme consistency.

Estimated Code Review Effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly Related PRs

Poem

🐰 With tokens so new, the colors take flight,
From darkness of white to a semantic light,
Each demo now shines with consistency bright,
Design systems dance in the rabbits' delight! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is entirely empty with only the template structure present. All required sections lack content: no description of changes, no related issues, no type of change selection, no testing details, and no additional notes. Fill in all required sections: describe what changes were made and why, link any related issues, select the appropriate type of change, explain testing performed, and add relevant notes about the theme support implementation.
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 (1 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Have the paywall support all themes' clearly and accurately describes the main change: updating paywall components to work with all themes by replacing hard-coded colors with semantic design tokens.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch paywall-light

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.

@saddlepaddle saddlepaddle merged commit 250d859 into main Jan 30, 2026
5 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

  • ✅ Neon database branch
  • ⚠️ Electric Fly.io app

Thank you for your contribution! 🎉

@Kitenite Kitenite deleted the paywall-light branch February 5, 2026 01:35
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