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

Ensure visibility of buttons and radio/checkbox for dark primary color in dark theme just like we do for light primary color in default light theme #2661

Closed
jancborchardt opened this issue Apr 26, 2022 · 4 comments
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working design Design, UX, interface and interaction design

Comments

@jancborchardt
Copy link
Contributor

As a follow-up to #2622 by @skjnldsv. We need a fallback for the case of dark primary color in dark theme just like we do for bright primary color in light theme.

Light theme + bright primary colour

This looks good – here we offer a fallback to a grey tone so there is enough contrast with the background, and the primary button looks more present than the secondary one:
Peek 08-04-2022 10-19

Dark theme + black primary colour

Here the radio buttons and checkboxes are barely visible, and the primary button is way less visible than the secondary one:
Peek 08-04-2022 10-16

@jancborchardt jancborchardt added bug Something isn't working 1. to develop Accepted and waiting to be taken care of design Design, UX, interface and interaction design labels Apr 26, 2022
@skjnldsv
Copy link
Contributor

skjnldsv commented Apr 26, 2022

Thank you for opening this, it was on my todo!
Do you have an idea on an already existing variable we should use for the icon colour then?

@jancborchardt
Copy link
Contributor Author

The only variable which comes close is text-maxcontrast but that would be misusing it.

I guess we need color-primary-element-dark in addition? Or can color-primary-element know if it needs to be triggered for dark themes?

@skjnldsv
Copy link
Contributor

I guess we need color-primary-element-dark in addition? Or can color-primary-element know if it needs to be triggered for dark themes?

It could. We just have to define clear expectations so I know what I can put :)

@susnux
Copy link
Contributor

susnux commented Jun 26, 2023

Fixed by @szaimen in #4067

On this screen recording the primary color was set to black:

vokoscreenNG-2023-06-26_15-59-01.mp4

@susnux susnux closed this as completed Jun 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working design Design, UX, interface and interaction design
Projects
None yet
Development

No branches or pull requests

3 participants