Skip to content

feat(projects): add theme toggle to Projects page header#689

Open
nino-chavez wants to merge 1 commit intoOpenCut-app:mainfrom
nino-chavez:feat/theme-toggle-projects-646
Open

feat(projects): add theme toggle to Projects page header#689
nino-chavez wants to merge 1 commit intoOpenCut-app:mainfrom
nino-chavez:feat/theme-toggle-projects-646

Conversation

@nino-chavez
Copy link

@nino-chavez nino-chavez commented Jan 29, 2026

Summary

  • Added ThemeToggle component to Projects page header
  • Works on both desktop and mobile layouts
  • Allows users to switch themes without navigating away

Related Issue

Fixes #646

Test plan

  • Theme toggle visible in desktop header
  • Theme toggle visible in mobile header
  • Click toggles between light/dark mode
  • Playwright E2E test included

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features
    • Added theme toggle functionality to the Projects page, accessible in both mobile and desktop layouts for convenient light/dark mode switching.

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

Added ThemeToggle component to the Projects page header for consistency
with the editor and landing pages. Toggle appears in both desktop and
mobile views next to the action buttons.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Jan 29, 2026

@nino-chavez is attempting to deploy a commit to the OpenCut OSS Team on Vercel.

A member of the Team first needs to authorize it.

@netlify
Copy link

netlify bot commented Jan 29, 2026

👷 Deploy request for appcut pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 6399c22

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 29, 2026

📝 Walkthrough

Walkthrough

The Projects page now includes a ThemeToggle component in both the mobile and desktop header sections. The component is rendered alongside existing action buttons without modifying any business logic or data handling.

Changes

Cohort / File(s) Summary
Theme Toggle Integration
apps/web/src/app/projects/page.tsx
Added ThemeToggle component import and rendering in both narrow (mobile) and wide (desktop) header areas alongside existing CreateButton and Select controls.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A toggle for themes, so bright and so fair,
Now Projects page shines with style everywhere,
Light mode, dark mode, the choice is all yours,
Consistency hops through all the app's doors!

🚥 Pre-merge checks | ✅ 4 | ❌ 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 (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding a theme toggle component to the Projects page header.
Description check ✅ Passed The description is comprehensive and covers all key aspects of the change, though it deviates from the repository template structure.
Linked Issues check ✅ Passed The PR fully addresses issue #646 by adding the ThemeToggle component to the Projects page header in both desktop and mobile layouts as required.
Out of Scope Changes check ✅ Passed All code changes are directly related to the objective of adding a theme toggle to the Projects page header with no unrelated modifications.

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

✨ Finishing touches
  • 📝 Generate docstrings

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.

nino-chavez added a commit to nino-chavez/OpenCut that referenced this pull request Jan 31, 2026
- Add theme toggle to Projects page header (OpenCut-app#689)
- Fix nested button HTML validation errors in header and hero (OpenCut-app#687)
  - Use asChild prop to render Links with button styles
- Add ability to add multiple keyboard shortcuts per action (OpenCut-app#696)
  - New "+" button to add additional shortcuts
  - Replace mode only removes the specific key being edited
  - Right-click to remove a shortcut when multiple exist

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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.

[FEATURE] Add theme toggle button to Projects page

1 participant