After ‘arcto’ path command(a or A), spaces are no longer removed by default

[yisibl]

Background

In the svg elliptical arc curve commands, the white space after large-arc-flag and sweep-flag can be removed. Svgo was implemented here(258fecf) before and enabled by default.

The idea of the implementer @GreLI at the time was:

It's a totally standard syntax from a 8-years old specification (and was introduced in a working draft a year before). If some software has a problems with it—those should be addressed to that software.

However, more and more people report that there will be problems. I think that as an svg compressor, security and compatibility are the most important. Although browsers all support omitting the white space after large-arc-flag and sweep-flag, mainstream vector software such as Sketch and Adobe Illustrator do not support it. Even if these software are supported, they are paid software, and the upgrade cost for users is huge. As a designer, I can't understand why the SVG file compressed by svgo will make an error when reopened. We can't think about the problem from the perspective of a developer who can write code. Even for a developer, the rules of path data are obscure.

So this PR disables the noSpaceAfterFlags option by default. Although the compressed file size increases slightly, I think it is worth it. If your SVG file is only used in the browser or pursues the ultimate file size, you can enable it through the following options:

{
    "plugins": [
        { "convertPathData": { "noSpaceAfterFlags": true } },
        { "mergePaths": { "noSpaceAfterFlags": true } }
    ]
}

Fixed: #1317 #1138 #1164 #1183 #1189

[yisibl]

@TrySound PTAL

[TrySound]

Thanks a lot @yisibl!

[Jupi007]

Thanks @yisibl 👍

[yisibl]

@TrySound Can you help me change the git email and username of the master branch commit? I used the wrong company email. Or I resubmit a PR. Thanks a lot.

[TrySound]

Sorry, don't want to rewrite master history especially after release. Looks like both are here. Is it a big problem?

[yisibl]

You are so fast! That's it for now.

[GreLI]

It's a fame, that software doesn't support SVG well 😒