Skip to content

use PatternFly light, dark, and system themes - #76

Merged
jkandasa merged 1 commit into
mycontroller-org:mainfrom
jkandasa:remove-custom-theme
Sep 19, 2026
Merged

jkandasa merged 1 commit into
mycontroller-org:mainfrom
jkandasa:remove-custom-theme

Conversation

@jkandasa

Copy link
Copy Markdown
Member

Summary

Replace data-repository custom GUI themes (labels.gui_theme) with PatternFly light, dark, and system.

  • Theme is stored in the browser as localStorage key mc-theme and applied before React paints.
  • Header (left to right): compact joined theme group (desktop / sun / moon) → locale (🇬🇧 EN-UK) → notifications → help → user.
  • Locale menu: flag + name, with EN-UK / TA-IN as secondary text. Light-theme list uses explicit dark text so names are visible.
  • Labels, forms, topology, and widgets follow PatternFly colour tokens in dark mode.
  • Dashboard widget tiles no longer have a grey fill.

Test plan

  • Header theme group: System / Light / Dark. Selected button is solid blue on the dark bar.
  • Refresh after choosing Dark; page stays dark (localStorage.mc-theme === "dark").
  • System follows OS colour scheme and updates if the OS changes.
  • Locale dropdown shows 🇬🇧 EN-UK; list shows names in light theme (not white-on-white).
  • Details page labels: pale-blue key, solid-blue value with white text (light); readable chips in dark.
  • Dashboard has no grey canvas or grey widget fill.
  • Login page follows the same theme.

Remove custom data-repository GUI themes. Persist the choice in
localStorage (mc-theme). Header has a compact joined theme
control and a locale dropdown that shows the flag plus EN-UK.
Dark mode uses PatternFly tokens for labels, forms, and widgets.
Dashboard tiles no longer use a grey fill.
@jkandasa
jkandasa merged commit 3eb453c into mycontroller-org:main Sep 19, 2026
1 check passed
@jkandasa
jkandasa deleted the remove-custom-theme branch September 19, 2026 04:12
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