Skip to content

Fix theme cycling wiping all CSS#35

Merged
shranchi0 merged 1 commit intoAshirBorah:mainfrom
shranchi0:fix/ui-polish
Feb 18, 2026
Merged

Fix theme cycling wiping all CSS#35
shranchi0 merged 1 commit intoAshirBorah:mainfrom
shranchi0:fix/ui-polish

Conversation

@shranchi0
Copy link
Collaborator

Summary

  • Theme cycling (Ctrl+T) called stylesheet.source.clear() which nuked the app's structural layout CSS (#main-content, #right-panel sizing), collapsing the UI to a single colored screen
  • TAMEApp.CSS had hardcoded Screen { background; color } that overrode theme colors even when themes tried to set them
  • Now: Screen colors come solely from theme files, and structural layout CSS is preserved across theme switches

Follow-up to #34.

Test plan

  • Open app, cycle through themes with Ctrl+T — layout should stay intact, colors should change

action_toggle_theme called stylesheet.source.clear() which removed
the app's structural layout CSS (#main-content, #right-panel sizing).
Only the theme file was re-added, so the UI collapsed to a single
colored screen.

- Remove hardcoded Screen colors from TAMEApp.CSS so themes control it
- Re-add app's structural CSS after clearing before adding theme CSS
@shranchi0 shranchi0 merged commit af38509 into AshirBorah:main Feb 18, 2026
5 checks passed
@shranchi0 shranchi0 deleted the fix/ui-polish branch February 18, 2026 19:59
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

Comments