xmlns attribute is required for single SVG file

[marmooo]

xmlns attribute is required for single SVG file

[tdewolff]

Good point, this should be fixed in the latest release!

[tdewolff]

Thanks for raising this issue. The XMLNS attribute should only be removed when the SVG document is "inline". This is only activated for SVG elements inside HTML. Otherwise, a regular SVG document is not considered inline and should thus not remove the XMLNS attribute. It is strange that this does happen for you, could you show me an example that reproduce this behaviour?

[marmooo]

Sorry for the lack of examples.
After further investigation, it appears to be a random bug.
It seems to happen about 70% of the time.
Since it was not possible to check without a certain number of files,
I will provide an example of a repository where the error occurs.

go install github.com/tdewolff/minify/v2/cmd/minify@latest
git clone https://github.com/marmooo/touch-kanji
cd touch-kanji
minify src/favicon/favicon.svg  # --> ok
minify -r docs -o .  # --> sometimes fail

The easiest way to check is to enter the following commands:

cd touch-kanji
minify -r docs -o .
git status  # <-- if docs/favicon/favicon.svg is added, it's a bug.