-
Notifications
You must be signed in to change notification settings - Fork 46
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
RFC: Focus and hover styles on ContextMenuItems #1073
Comments
@TejasQ would your proposed changes effecting all dropdown regardless of where they're being used? |
Yes. Please be aware that this is how all |
@TejasQ I like the subtleness, but I would prefer to keep what's already being designed (the new one). One of the main reason is that we have this combo-dropdown (with custom text field) - look at the background color of the "custom": The other reason is that I am deliberately making our focus and hover state a bit "catchy", so that it is more usable (think of users that don't have such a good eyes like ours ;)). |
I hear what you're saying, but bigger list items look a little weird: see the second example here. What do we do in this case? |
Is it weird because of its narrow width you mean? |
Weird because it's super blue there but not super blue here even though they're conceptually the same thing. |
@kemal-contiamo what conclusion did we come to here? |
I feel the current style of focus and hover of ContextMenuItem is a little too out of place. It's bright and blue and attracts too much attention.
For these reasons, I propose restyling the hover/focus states to be the following:
Hover
Focus
If we agree, I'll go ahead and open a PR with changes that implement this adjustment.
RFC @kemal-contiamo @micha-f @contiamo/frontend
The text was updated successfully, but these errors were encountered: