search

ben-eb / gulp-svgmin

Minify SVG files with gulp.
MIT License
341 stars 35 forks source link

Should elements be merged and classes preserved? #103

Closed ronmurp closed 3 years ago

ronmurp commented 3 years ago

Comparing gulp-svgmin with grunt-svgmin, Expected a and enclosed to be merged into a with both classes, as in grunt-svgmin@3.3.0 example.

Original: ``

``

grunt-svgmin@3.3.0 merges g & rect into a path and combines class values ``

``

gulp-svgmin@3.0.0 & grunt-svgmin@6.0.1 leave the g & rect unmerged ``

``

This issue was observed on larger project files, converting a legacy grunt project to a gulp project. The test sample above was used to illustrate the issue.

ben-eb commented 3 years ago

Hey @ronmurp, please could you report this to https://github.com/svg/svgo if you are still having this issue? This plugin is a thin wrapper on top of svgo to use it with gulp, and so this is better reported upstream. Thanks. 👍