-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
There's no way to edit SVG files in the project because they are "overcompressed" #6334
Comments
Well done! We should call our compressor |
Related svg/svgo#1164 and svg/svgo#1137. |
Looks like the following helps: diff --git a/scripts/svgo.config.json b/scripts/svgo.config.json
index a5ba4161..dd238555 100644
--- a/scripts/svgo.config.json
+++ b/scripts/svgo.config.json
@@ -3,8 +3,9 @@
"collapseGroups",
"removeDimensions",
{ "removeAttrs": { "attrs": "(fill|stroke|fill-rule)" } },
+ { "convertPathData": { "noSpaceAfterFlags": false } },
"removeTitle",
"removeComments",
"removeMetadata"
]
-}
\ No newline at end of file
+} |
I can confirm. Fix works fine! 🎉 (tested locally) |
I created a PR. It solves the technical aspect and brings mangled icons back to life but we may not be that lucky next time. Let's discuss the way we store, share and develop editor resources like Sketch files. |
Internal: The SVG icons compressed by the clean-up-svg-icons task (svgo) should be editable in graphical editors. Closes #6334.
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
…go) should be editable in graphical editors (see ckeditor/ckeditor5#6334)
Re-opening because of
|
Are you sure? This is a good followup to the issue. I would close it and open a new one. |
Alright, please a new one and put some reference to this one. |
Provide a description of the task
Check out what
link.svg
looks like when opened in Sketch:(similar results in other graphic editors).
This is because of our aggressive
clean-up-svg-icons
task squeezes every byte of the SVG files using every technique available and SVG editors don't handle this well on the input (FYI it looks ok in the browser).Unless we have these icons stored somewhere locally (or retrieved from the git history) there's no way to edit and improve them. Or copy and use as a base for future icons.
We need to either:
cc @panr @dkonopka @Reinmar
The text was updated successfully, but these errors were encountered: