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

New Octicons don't have enough contrast #79278

Closed
miguelsolorio opened this issue Aug 16, 2019 · 7 comments
Closed

New Octicons don't have enough contrast #79278

miguelsolorio opened this issue Aug 16, 2019 · 7 comments
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s) icons-product Issues for in-product icons

Comments

@miguelsolorio
Copy link
Contributor

miguelsolorio commented Aug 16, 2019

Related #76909

Old

image

New

image

Bold

image

This feedback is not around the shape of the icon but the fact that our current octicons all have a solid background applied which puts a very nice contrast on top of the status bar. With the new transparent icons we are loosing this contrast. They almost look like the reverse icon of what we have today (e.g. now transparent where we used to have white background).

Originally posted by @bpasero in #79234 (comment)

@miguelsolorio miguelsolorio self-assigned this Aug 16, 2019
@miguelsolorio miguelsolorio added the icons-product Issues for in-product icons label Aug 16, 2019
@pedily
Copy link

pedily commented Aug 16, 2019

I think when they were implemented they were designed and tested on high-dpi screens.
For more "traditional" setups like 24" 1080p, they look blurry because they get antialiased to transparency.
Screenshot from 2019-08-16 08-58-45

@miguelsolorio
Copy link
Contributor Author

@strafbier this issue is about Octicons in the status bar and quick pick, not the activity bar.

See this comment about why they appear blurry.

@miguelsolorio miguelsolorio changed the title New octicons don't have enough contrast New Octicons don't have enough contrast Aug 16, 2019
@ErikHumphrey
Copy link

The solid icons would be fine if they had the same positioning/height/spacing as New and Bold, rather than appearing as off-center or not taking up enough of the space on the status bar.

@sstaub
Copy link

sstaub commented Sep 7, 2019

The new icons are horrible designed and not helpful for perception.

@miguelsolorio
Copy link
Contributor Author

@sstaub can you elaborate? Are you referring to our new icons everywhere or just the Octicons?

@sstaub
Copy link

sstaub commented Sep 10, 2019

Both, the icons on the sidebar also on the statusbar (insider version).
The buttons and symbols on statusbar and sidebar were better before, because they had a better contrast.
Maybe we can choose between old and new design in settings.

@miguelsolorio
Copy link
Contributor Author

Thanks everyone for the feedback. In trying to keep our issue count on a manageable level, I'm going to close this issue as a duplicate of #79130 since that will solve this issue. Please upvote the issue there.

@miguelsolorio miguelsolorio added the *duplicate Issue identified as a duplicate of another issue(s) label Oct 7, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s) icons-product Issues for in-product icons
Projects
None yet
Development

No branches or pull requests

4 participants