-
Notifications
You must be signed in to change notification settings - Fork 29.9k
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
Update toolbar actions with hover feedback #120753
Comments
Also worth looking at the activity bar as well, but that seems like it needs more consideration as we have the active indicator and active background color token cc @sandy081 |
@misolori hover backgrounds in activity bar and panel are handled here - #118077 (comment) |
@misolori @sandy081 After playing around with it today, I'm not entirely convinced that the hover background color on activity actions and panel titles works as well as it does in tiny 16px actions. I would simply keep a foreground hover change on those, what do you think? Statusbar looks good tho. |
Removed hover background until we come to a conclusion. |
I liked the hover background with custom hovers. I felt they go well along together. |
I like the hover feedback on toolbar actions a lot, but feel they are not so nice in activity bar and panel given we already have hover feedback there for a long time. I would decouple the idea of custom hovers there from the hover feedback imho. You can do custom hovers but keep the current hover feedback. |
My opinion copied from Slack is almost @bpasero's verbatim:
|
Closing in favor of #118077 |
Now that #120226 has added hover feedback, we should look at improving the other areas that use this to make them feel consistent. Here's an idea to update the notebook toolbar and status bar items to also have matching corners:
We'd need to account for when items in the status bar that have a bg color (like the remote indicator or error bg)
cc @bpasero for status bar, @rebornix for notebooks
The text was updated successfully, but these errors were encountered: