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

Update toolbar actions with hover feedback #120753

Closed
miguelsolorio opened this issue Apr 7, 2021 · 9 comments
Closed

Update toolbar actions with hover feedback #120753

miguelsolorio opened this issue Apr 7, 2021 · 9 comments
Assignees
Labels
notebook ux User experience issues workbench-status Status bar

Comments

@miguelsolorio
Copy link
Contributor

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:

CleanShot 2021-04-07 at 07 47 33

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

@miguelsolorio miguelsolorio added ux User experience issues workbench-status Status bar notebook labels Apr 7, 2021
@miguelsolorio miguelsolorio self-assigned this Apr 7, 2021
@miguelsolorio
Copy link
Contributor Author

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

@sandy081
Copy link
Member

@misolori hover backgrounds in activity bar and panel are handled here - #118077 (comment)

@joaomoreno
Copy link
Member

joaomoreno commented Apr 19, 2021

@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.

@joaomoreno joaomoreno added this to the April 2021 milestone Apr 19, 2021
@sandy081
Copy link
Member

Removed hover background until we come to a conclusion.

@sandy081 sandy081 removed their assignment Apr 20, 2021
@sandy081 sandy081 removed this from the April 2021 milestone Apr 20, 2021
@sandy081
Copy link
Member

I liked the hover background with custom hovers. I felt they go well along together.

@bpasero
Copy link
Member

bpasero commented Apr 20, 2021

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.

@joaomoreno
Copy link
Member

joaomoreno commented Apr 22, 2021

My opinion copied from Slack is almost @bpasero's verbatim:

there are two distinct UI features here that I wish we decouple asap:

  • tooltips
  • hover background color on activity and panel

the tooltips are great for discoverability: that should be behind a setting. but the hover background color on activity and panel gives a heaviness to the workbench: that should just be reverted. when I added hover background color to actions I tried activity bar and panel and did not push that on purpose, since it looks really odd. please consider just removing this @sandy081

@miguelsolorio
Copy link
Contributor Author

Closing in favor of #118077

@github-actions github-actions bot locked and limited conversation to collaborators Jun 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
notebook ux User experience issues workbench-status Status bar
Projects
None yet
Development

No branches or pull requests

5 participants
@joaomoreno @bpasero @sandy081 @miguelsolorio and others