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

RFC: Focus and hover styles on ContextMenuItems #1073

Open
TejasQ opened this issue Jun 23, 2019 · 7 comments
Open

RFC: Focus and hover styles on ContextMenuItems #1073

TejasQ opened this issue Jun 23, 2019 · 7 comments
Assignees

Comments

@TejasQ
Copy link
Contributor

TejasQ commented Jun 23, 2019

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.

image

For these reasons, I propose restyling the hover/focus states to be the following:

Hover

image

Focus

image

If we agree, I'll go ahead and open a PR with changes that implement this adjustment.

RFC @kemal-contiamo @micha-f @contiamo/frontend

@TejasQ TejasQ self-assigned this Jun 23, 2019
@kemal-contiamo
Copy link

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.

image

For these reasons, I propose restyling the hover/focus states to be the following:

Hover

image

Focus

image

If we agree, I'll go ahead and open a PR with changes that implement this adjustment.

RFC @kemal-contiamo @micha-f @contiamo/frontend

@TejasQ would your proposed changes effecting all dropdown regardless of where they're being used?

@TejasQ
Copy link
Contributor Author

TejasQ commented Jun 24, 2019

Yes. Please be aware that this is how all
dropdowns looked before.

@kemal-contiamo
Copy link

@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":
Screenshot 2019-06-24 at 09 28 06

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 ;)).

@TejasQ
Copy link
Contributor Author

TejasQ commented Jun 24, 2019

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?

@kemal-contiamo
Copy link

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?
Screenshot 2019-06-24 at 10 05 37
or weird because the way focus and hover state looks in those menu?

@TejasQ
Copy link
Contributor Author

TejasQ commented Jun 24, 2019

Weird because it's super blue there but not super blue here even though they're conceptually the same thing.

@TejasQ
Copy link
Contributor Author

TejasQ commented Jun 25, 2019

@kemal-contiamo what conclusion did we come to here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants