Skip to content

feat(admin): add theme preferences and refine dark mode styling#10

Merged
coding-hui merged 1 commit into
mainfrom
fix/admin-user-theme-switcher
May 17, 2026
Merged

feat(admin): add theme preferences and refine dark mode styling#10
coding-hui merged 1 commit into
mainfrom
fix/admin-user-theme-switcher

Conversation

@coding-hui
Copy link
Copy Markdown
Owner

Summary

  • Add a real admin theme switcher with light, dark, and system modes, defaulting to system preference.
  • Remove the developer SettingDrawer from the runtime UI and disable MFSU to avoid stale module federation runtime failures during local dev.
  • Normalize admin status tags, page descriptions, buttons, links, and account settings styling across light and dark themes.
  • Add focused tests for theme preference normalization and subscriptions.

User Impact

Admins can switch themes from the top header without seeing development-only controls. Dark mode now has readable primary/default buttons, table actions, status tags, page descriptions, and account settings surfaces while preserving the light theme appearance.

Validation

  • pnpm --dir apps/admin biome
  • gofmt -w $(rg --files -g '*.go')
  • pnpm --dir apps/admin lint
  • pnpm --dir apps/admin jest --runInBand
  • pnpm --dir apps/admin build
  • go test ./...
  • Chrome regression checks for light, dark, and system theme modes on admin pages

Note: admin build still reports the existing Browserslist/caniuse-lite freshness warning; build completes successfully.

@coding-hui coding-hui changed the title [codex] Improve admin theme switching Add admin theme preferences and refine dark mode styling May 16, 2026
@coding-hui coding-hui changed the title Add admin theme preferences and refine dark mode styling feat(admin): add theme preferences and refine dark mode styling May 16, 2026
@coding-hui coding-hui marked this pull request as ready for review May 16, 2026 16:04
@coding-hui coding-hui merged commit 84e89c4 into main May 17, 2026
2 checks passed
@coding-hui coding-hui deleted the fix/admin-user-theme-switcher branch May 17, 2026 02:09
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