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
Improved rendering of Seti icons #22667
Comments
Digging a bit further, I assume you've used the seti.woff file from the atom-seti-icons project, which in turn takes them from the seti-ui project. Seti UI uses a script that converts the good-looking SVG icons into font files, and it appears to be there that the problem occurs. This combines the icons and outputs them into five font files (seti.eot, seti.svg, seti.ttf, seti.woff, and seti.woff2). Both the TTF and WOFF files have the disfigured glyphs. I don't have tools to open the WOFF2 and EOT formats directly. The seti.svg font file opens in FontForge and shows most of the icons correctly -- CSV and Shell are still missing their detail. So, ultimately, this seems to be an issue with the conversion of the source SVG icons into a font-file. |
This issue was moved to jesseweed/seti-ui#401 |
I've got a pull request in with Seti UI to resolve this issue -- fix for VS Code I expect will be an updated seti.woff and changes to vs-seti-icon-theme.json to account for some additional icons. With Atom, I was able to just drop the new fonts into the theme's package folder and everything seems to work, but with VSCode, I get this strangeness: Updated font can be found in the pull request if you wish to test for yourself: jesseweed/seti-ui#406 |
Ignore the above. The mis-alignment was due to an error in one of the source icons that was throwing off the font size normalisation. Seems to all be working in Atom now, and initial tests with VSCode look promising. My assumption is that you'll pull the updated WOFF from the Seti UI project once it's merged. A small preview below :) |
@gmckeown Looks great, thanks for your work! I hope your changes get adopted soon so I can pick them up. |
@gmckeown I updated the Seti font containing your fixes. Thanks a lot! |
Good stuff! If you do see any vertical alignment issues with the updated font -- per #22667 (comment) above -- there's a separate pull request to address this: jesseweed/seti-ui#409. When the Insiders build updates to include your commit I'll test on the machine that had the issue before. |
Yes, I had to manually patch seti-ui to get the vertical alignment correct on windows. Thanks for creating a pull request for that! |
@aeschli Today's Insider build is looking great on the Win10 notebook on which I previously saw the alignment issues :). I'll add a comment here if/when my fix is pulled into Seti-UI. |
Sorry, just realised I left myself logged in to an old account I was in the process of retiring! Previous comments were from me :) |
@aeschli, I don't see mention of the updated icons in current Insiders release notes -- will this make it into 1.12.0? |
It's in there, sorry. I'll add a note. |
I added a thank you note to the release notes! Thanks again @gmckeown ! |
Awesome, cheers! |
Steps to Reproduce:
Icon is simply a grey square; the Seti icon set for Atom suggests there should be some detail displayed.
Here's an enlarged comparison with VSCode on top and the version from the Seti summary below:
And here's a rendering of the original Seti shell.svg using Inkscape:
Loading the seti.woff file (in eval version of FontCreator) shows the icons to be poorly rendered at larger sizes as well, which I assume is due to a problem converting from the SVGs to WOFF?
A selection of the icons as viewed in FontCreator:
Here's the C# Icon from the WOFF:
And here's the same icon from the original SVG (viewed in Inkscape):
The text was updated successfully, but these errors were encountered: