Skip to content
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

Custom color support leads to weird UI rendering #2661

Open
dnadlinger opened this issue Jan 29, 2025 · 2 comments
Open

Custom color support leads to weird UI rendering #2661

dnadlinger opened this issue Jan 29, 2025 · 2 comments
Assignees

Comments

@dnadlinger
Copy link
Collaborator

dnadlinger commented Jan 29, 2025

Even without any custom colors set, the custom color support added in 52c07a2 interferes with normal widget styling, causing odd-looking results.

One commit before 52c07a2:
Image

Current master (0f0f030):
Image

This is my local dev box with macOS in particular, but I imagine the same will happen on all sorts of other platforms (e.g. dark mode?). Generally, interfering with widget styling in an "always-on" fashion seems incredibly brittle – sb10q found something similar on his (presumably Linux) box on an earlier version of #2590. Leaving the default styling alone if no colours are set seems like the much better way, unless we want to test all sorts of OS/theming combinations.

@sbourdeauducq
Copy link
Member

Note that macOS is not a supported platform.

But it does seem that e.g. the gradients were broken on all platforms by #2590. And the gradients still should be there even with the colors applied.

@dnadlinger dnadlinger changed the title Custom color support leads to werid UI rendering Custom color support leads to weird UI rendering Jan 29, 2025
@dnadlinger
Copy link
Collaborator Author

dnadlinger commented Jan 29, 2025

Yes, I am by no means complaining about macOS specifically – it just supports my broader point that messing with the default OS rendering "without reason" (on the default path of a somewhat niche optional feature) may be more brittle than we should be happy with.

For instance, here is another problem on Windows (the spin box field is grey, almost as if it were disabled):

Image

Just not setting any custom styles at all when no custom colour is configured would avoid issues like this entirely.

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

No branches or pull requests

3 participants