feat(ui): add theme toggle and clean up loading state - #175
Merged
Conversation
Add a compact light/dark control to authenticated and sign-in surfaces while preserving the system color scheme as the first-visit default. Persist an explicit selection and keep system changes in sync until the user chooses a mode. Remove the raised panel, gradient, border, and shadow treatments from loading states so the branded animation sits directly on the page background in either theme.
HazAT
marked this pull request as ready for review
August 27, 2026 10:52
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 9729813. Configure here.
Resolve the saved or system theme in a parser-blocking head script so the document receives its theme attribute before the generated stylesheet loads. Remove the deferred module initializer, while leaving the interactive toggle responsible for subsequent theme changes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Loading screens now place the branded animation directly on the page canvas instead of inside a darker raised panel or decorative gradient. A compact theme control is available in the main header and on sign-in and loading screens so users can switch between light and dark mode from every application state.
The first visit continues to follow the operating-system preference, with the initial theme applied before the stylesheet loads to avoid a mismatched first paint. Once a user chooses a mode, that selection is remembered; system theme changes remain live only until an explicit choice is made.