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

Add setting to toggle new octicon style #79234

Merged
merged 2 commits into from
Aug 16, 2019
Merged

Conversation

miguelsolorio
Copy link
Contributor

This is to begin the work of #76909 where we need to update Octicons to match our new icon style. I've introduced a new hidden setting, workbench.octiconsUpdate.enabled, that when set to true will enable the new version. This will allow us to better selfhost and test changes. Once the team feels confident about the new icons we can remove this and make it the default.

image

To test the Octicons, you can use this extension that I created that toggles all of the icon options in the status bar (make sure you have a file open and a wide width for them to show up).

Once this is merged in we'll ask the team to selfhost and create issues for icon changes/modifications. A good portion of these are from our existing icon library. I also increased the icon font to 16px in the status bar (from 14px) since these icons are smaller.

I'll be out next week so there's a good chance this will carry over into the next milestone.

@miguelsolorio miguelsolorio added this to the August 2019 milestone Aug 15, 2019
@miguelsolorio miguelsolorio requested a review from bpasero August 15, 2019 16:33
@miguelsolorio miguelsolorio added the icons-product Issues for in-product icons label Aug 15, 2019
@bpasero
Copy link
Member

bpasero commented Aug 16, 2019

Not sure if you want feedback already, but as I mentioned earlier, the new icons do not work for me in the status bar.

Old:
image

New:
image

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

@miguelsolorio
Copy link
Contributor Author

Ok, I'll merge this and will create an issue to track that. Not sure what else we can besides create another solid version but that doubles the work. Let's see what others think, should we turn this on as a default for a week and then turn off before endgame?

@miguelsolorio miguelsolorio merged commit 67c59a2 into master Aug 16, 2019
@miguelsolorio miguelsolorio deleted the misolori/octicon-update branch August 16, 2019 06:45
@bpasero
Copy link
Member

bpasero commented Aug 16, 2019

@misolori

should we turn this on as a default for a week and then turn off before endgame?

Yes, to get most feedback that makes sense.

@bpasero
Copy link
Member

bpasero commented Aug 16, 2019

Not sure what else we can besides create another solid version but that doubles the work

Can we play with CSS to get there maybe?

@miguelsolorio
Copy link
Contributor Author

Can we play with CSS to get there maybe?

Yea, I'll play around with the weights to see if there's a way to achieve more contrast.

@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
icons-product Issues for in-product icons
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants