Skip to content

feat(ui): add theme toggle and clean up loading state - #175

Merged
HazAT merged 2 commits into
masterfrom
feat/theme-toggle-loader-background
Aug 27, 2026
Merged

feat(ui): add theme toggle and clean up loading state#175
HazAT merged 2 commits into
masterfrom
feat/theme-toggle-loader-background

Conversation

@HazAT

@HazAT HazAT commented Aug 27, 2026

Copy link
Copy Markdown
Member

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.

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
HazAT marked this pull request as ready for review August 27, 2026 10:52

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ 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.

Comment thread src/app/styles.css
Comment thread src/app/theme.ts
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.
@HazAT
HazAT merged commit 49f883d into master Aug 27, 2026
13 checks passed
@HazAT
HazAT deleted the feat/theme-toggle-loader-background branch August 27, 2026 11:33
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