Skip to content

FEATURE: theme default option in user interface #34110

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Aug 7, 2025
Merged

FEATURE: theme default option in user interface #34110

merged 2 commits into from
Aug 7, 2025

Conversation

lis2
Copy link
Contributor

@lis2 lis2 commented Aug 6, 2025

Before the "theme default" option appeared conditionally, which caused confusion. For example, it was shown when the theme was using a color scheme that was not user-selectable. If the scheme was selectable, then a specific scheme like "Merigold" was preselected. If the theme changed default schemes, that change was not reflected on the user interface.

Therefore, it would be better to always have the "Theme default" option, which would have -1 id. It means that the user's color scheme will always follow theme defaults.

Screenshot 2025-08-06 at 10 07 40 am

@lis2 lis2 force-pushed the scheme-interface branch from 107e47f to 252a831 Compare August 6, 2025 02:21
@lis2 lis2 changed the title REFACTOR: simplify color scheme interface and default handling FEATURE: theme default option in user interface Aug 6, 2025
@lis2 lis2 force-pushed the scheme-interface branch from 252a831 to a0cb320 Compare August 6, 2025 02:24
Before the "theme default" option appeared conditionally, which caused confusion. For example, it was shown when the theme was using a color scheme that was not user-selectable. If the scheme was selectable, then a specific scheme like "Merigold" was preselected. If the theme changed default schemes, that change was not reflected on the user interface.

Therefore, it would be better to always have the "Theme default" option, which would have `-1` id. It means that the user's color scheme will always follow theme defaults.
@lis2 lis2 force-pushed the scheme-interface branch 2 times, most recently from 7460da4 to a58d946 Compare August 6, 2025 04:35
@lis2 lis2 force-pushed the scheme-interface branch from a58d946 to 07e210e Compare August 6, 2025 05:08
@lis2 lis2 merged commit 4a604f0 into main Aug 7, 2025
16 checks passed
@lis2 lis2 deleted the scheme-interface branch August 7, 2025 01:05
lis2 added a commit that referenced this pull request Aug 7, 2025
lis2 added a commit that referenced this pull request Aug 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants