should keep unicode space char

[jaylecn]

input

<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="400" height="500">
<text fill="#000000" stroke="none" font-family="sans-serif" font-size="10px" font-style="normal" font-weight="normal" text-decoration="normal" x="20" y="20" text-anchor="start" dominant-baseline="alphabetic">
test&#160;spaces&#160;&#160;&#160;😑&#160;&#160;&#160;&#160;&#160;&#160;&#160;&#160;&#160;&#160;&#160;end
</text></svg>

output

<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="400" height="500"><text fill="#000000" stroke="none" font-family="sans-serif" font-size="10px" font-style="normal" font-weight="normal" text-decoration="normal" x="20" y="20" text-anchor="start" dominant-baseline="alphabetic">test spaces 😑 end</text></svg>

i want keep spaces, but even i use unicode space char, it's still removed, bug ?

[GreLI]

It's a parser issue, SVGO uses https://github.com/isaacs/sax-js parser, and as XML parser it eats spaces.

[jaylecn]

it's latest commit on 2017 😶 any solution? fix sax-js ?

[KrasnayaPloshchad]

Sax-js is no longer maintained after version 1.2.7 released, but I discovered a fork of this parser in their issues. This is under active maintaince, but has some differences than original one.
https://github.com/lddubeau/saxes

[strarsis]

@KrasnayaPloshchad: So it wouldn't be that hard to replace sax.js with saxes?
When it is more or less just a drop-in replacement I could give it a try.

PR: #1281

[KrasnayaPloshchad]

I think yes. But if there is regression bring by saxes itself that is impossible to fix by pull request, or saxes developer have no idea for what you need to fix, you can try other alternates anyway.

[strarsis]

Edit: The txml parser is an excellent candidate for which a PR is ready!

[SethFalco]

This was not an issue in sax, and rather was a configuration issue on our end. A contributor corrected this for us, and the fix was released in SVGO v2.1.0:

• 9f084cc#diff-4a0b83ac4f1aaf48931fcdbfc22b5a79bd10cc3a414e2b4a6ce22b622c4cf2feR13
• Preserve whitespace in elements containing text #1220

The optimizing the SVG works as expected in SVGO v3.2.0.