Skip to content

Fix command palette theme toggle to use NavbarContext - #23

Open
aayan-rashid wants to merge 1 commit into
NexGenStudioDev:masterfrom
aayan-rashid:fix-command-palette-theme
Open

Fix command palette theme toggle to use NavbarContext#23
aayan-rashid wants to merge 1 commit into
NexGenStudioDev:masterfrom
aayan-rashid:fix-command-palette-theme

Conversation

@aayan-rashid

Copy link
Copy Markdown

Summary

This PR updates the Command Palette theme toggle to use the shared NavbarContext instead of directly manipulating the DOM and localStorage.

Changes

  • Replaced direct theme manipulation with useNavbar().setTheme().
  • Removed duplicate theme management logic from CommandPalette.
  • Ensures all components consuming NavbarContext remain synchronized after toggling the theme.

Testing

  • Verified the project runs successfully with npm run dev.
  • Verified all existing tests pass (27/27).
  • Confirmed the theme toggles correctly through the Command Palette and stays synchronized with the navbar theme toggle.

@gemini-code-assist

Copy link
Copy Markdown

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@aayan-rashid
aayan-rashid force-pushed the fix-command-palette-theme branch from 0dbc19d to 3ddbe56 Compare July 27, 2026 06:07
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