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

Fix SVGO distortions #213

Merged
merged 2 commits into from
Apr 7, 2020
Merged

Fix SVGO distortions #213

merged 2 commits into from
Apr 7, 2020

Conversation

mdo
Copy link
Member

@mdo mdo commented Apr 7, 2020

Fixes #210.

According to svg/svgo#1164, some SVG renderers cannot display images correctly without a couple of configuration changes. This PR implements those changes and re-processes all the icons once more. Verified on a few icons with Sketch and Illustrator that they now open properly.

mdo added 2 commits April 7, 2020 12:38
SVGO options had to change in order to preserve an SVG's design and editability in other apps. Previously all icons rendered fine in browser, in macOS quick look, and in Figma, but not when opening directly in Sketch and Illustrator.
@mdo mdo added the bug Something isn't working label Apr 7, 2020
@mdo mdo added this to Inbox in v1.0.0-alpha4 via automation Apr 7, 2020
@mdo mdo merged commit c8b17d6 into master Apr 7, 2020
v1.0.0-alpha4 automation moved this from Inbox to Done Apr 7, 2020
@mdo mdo deleted the fix-svgo-distortions branch April 7, 2020 23:44
@mdo mdo mentioned this pull request Apr 8, 2020
@alstr
Copy link

alstr commented Nov 12, 2020

I am still noticing distortions when using the SVGs in Sketch. Not sure if I'm missing something.

Screenshot 2020-11-12 at 12 16 43

@mdo
Copy link
Member Author

mdo commented Nov 13, 2020

Hmm, yeah I think this is because we declare the width/height as 1em on each SVG. Those don't seem to be valid values in Sketch and more, so they show up as 1x1px instead of 16x16px. Might need to rethink that approach. 🤔 Going to open a new issue to track this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
v1.0.0-alpha4
  
Done
Development

Successfully merging this pull request may close these issues.

Can't open svg in Sketch & Illustrator
2 participants