-
Notifications
You must be signed in to change notification settings - Fork 492
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
Disable removeViewbox
plugin
#402
base: main
Are you sure you want to change the base?
Conversation
While svgo still has this enabled by default, it can cause issues
✅ Deploy Preview for svgomg ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Oh, and don't forget to read this for a lot more detail: svg/svgo#1128 |
People were complaining about this for many years but the sole owner of this project won't listen. |
It will be some time before we make the change in SVGO (v4 release), so I'd recommend merging this into SVGOMG already. It may also be worth adding either a warning, or including a link to the documentation for each plugin, so users are more informed on potential consequences of them. |
My bigger plan here is to extend each plugin with its individual options |
While svgo still has this enabled by default, it can cause issues
See svg/svgo#1461 for more details