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 Octicons to match new icon style #76909

Closed
miguelsolorio opened this issue Jul 8, 2019 · 21 comments · Fixed by #81505
Closed

Update Octicons to match new icon style #76909

miguelsolorio opened this issue Jul 8, 2019 · 21 comments · Fixed by #81505
Assignees
Labels
feature-request Request for new features or functionality icons-product Issues for in-product icons on-testplan

Comments

@miguelsolorio
Copy link
Contributor

miguelsolorio commented Jul 8, 2019

This is to track the work of updating Octicons to match our new outline icon style (#8017). We currently use Octicons in the Status Bar and Quick Pick items. Since we will be updating the icon style, this means that we will be replacing Octicons with our own custom icon font.

image

image

image

@miguelsolorio
Copy link
Contributor Author

This is now enabled by default in master (will be on Insiders next week). We've introduced a new settings workbench.octiconsUpdate.enabled that controls the visibility of the new Octicons style. Below is a snapshot of the updates:

image

@jletey
Copy link

jletey commented Aug 16, 2019

@misolori This is amazing! Can't wait to try out these new icons!

@jletey
Copy link

jletey commented Aug 18, 2019

@misolori Would it be possible to also have the Figma for the new octicons?

@miguelsolorio
Copy link
Contributor Author

@johnletey once they are finalized we will publish them as well to the same repo/Figma file. Though that might not happen until next month as we just added them and will be getting feedback/iterating on them.

@sean-mcmanus
Copy link
Contributor

Before:

image

After -- not what the C/C++ extension team wants:

image

Expected: Can you allow extensions to still use the old style somehow for their icons? Or do you have another suggestion?

@bobbrow
Copy link
Member

bobbrow commented Aug 20, 2019

I agree with @sean-mcmanus. The flame icon is unrecognizable at this point. Some outlines just don't work well enough to convey meaning.

@bobbrow
Copy link
Member

bobbrow commented Aug 20, 2019

Some icons got to keep their solid fills: image image image image image

Please keep the solid fill for the flame icon too.

@jletey
Copy link

jletey commented Aug 20, 2019

I'm a fan of the new GitHub logo

Shot 2019-08-20 at 17 12 12

@miguelsolorio
Copy link
Contributor Author

@sean-mcmanus @bobbrow if you are using Octicons then you won't be able to use the previous version, I'll see if we can improve the flame icon.

@miguelsolorio
Copy link
Contributor Author

⚠️ Update

Heads up that we're going to be disabling this as the default for Insiders tomorrow since we only had a week of coverage so it will move into the next iteration. You can still enable this by setting workbench.octiconsUpdate.enabled to true.

@miguelsolorio miguelsolorio modified the milestones: On Deck, September 2019 Aug 26, 2019
miguelsolorio pushed a commit that referenced this issue Aug 26, 2019
@bobbrow
Copy link
Member

bobbrow commented Aug 26, 2019

I'll see if we can improve the flame icon.

Thank you, @misolori!

@miguelsolorio
Copy link
Contributor Author

@sean-mcmanus @bobbrow here's a few alternates for the flame that match our outline style, let me know if you like one over the other. I personally prefer 1 or 2.

image

@sean-mcmanus
Copy link
Contributor

I'd prefer Flame 2 -- seems to match the original the closest.

Flame 1 and 3 don't look like flames to me -- too smooth and lacking "noise -- and searching the web for "flame icons" I get images that more closely resemble Flame 2 instead of Flames 1 or 3.

@eamodio
Copy link
Contributor

eamodio commented Aug 27, 2019

Flame 2 is my vote

@sean-mcmanus
Copy link
Contributor

sean-mcmanus commented Aug 27, 2019

Hmm...could the flame be made solid or can you make a solid-flame and outline-flame so users can choose which one they want? We color our flame red so it's more visible, but it's hard to tell if that will look as desired when it's an outline.

BTW, the Smiley face icon in the latest Insiders build looks strange (mouth seems too big).

@miguelsolorio
Copy link
Contributor Author

We’re trying to align all of our icons to be the same outline style so we don’t have plans to provide solid icons. We also don’t recommend to hard code any colors in the status bar icons as that is not themeable and introduces contrast issues.

@jletey
Copy link

jletey commented Aug 28, 2019

@misolori Personally like Flame 1

@bobbrow
Copy link
Member

bobbrow commented Aug 28, 2019

I like Flame 2 the best. To me, Flame 1 looks like a stylized "thumbs up" and Flame 3 looks like a teardrop.

I still think a more solid looking icon is better, but it could be my own bias against the thin lines. My eyesight is getting worse with time and most of these new icons look the same to me unless I really strain to differentiate them.

I apologize for defacing your art, but this is what my eyes understand best:
image

@Guema
Copy link

Guema commented Sep 2, 2019

@sean-mcmanus @bobbrow here's a few alternates for the flame that match our outline style, let me know if you like one over the other. I personally prefer 1 or 2.

image

The 1 is the best in my opinion.
Even if unfortunately a graphic chart can never satisfy everyone.
It's look like a flame, and it's simple.

2nd works fine, but more complex.
The line inside make it more readable in red on blue though.

3rd is simple but can create confusion as it also looks like a water drop.

model 1 has the most identifiable shape in my opinion, while 2 is more readable.
Maybe a 1 shaped with an inside shape like the 2 could do the trick ?
But for me the 1 is fine (maybe filled for color readability ?)

@Tyriar
Copy link
Member

Tyriar commented Sep 5, 2019

Flame 2 looks best to me, I thought Flame 1 was a thumbs up icon.

@miguelsolorio
Copy link
Contributor Author

Thanks everyone for the feedback, I've update the flame icon to version 2 and it'll be updated in tomorrow's Insiders:

image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality icons-product Issues for in-product icons on-testplan
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants