convertStyleToAttrs removes color

[JoKalliauer]

Processing File:2016_Angola_and_DR_Congo_yellow_fever_outbreak.svg with default options removes the color of countries:

svgo -i Bug1040_Input2.svg.txt -o Bug1040_Output2.svg.txt --pretty --indent=1 --enable=convertStyleToAttrs

Input

Bug1040_Input2.svg.txt

Output

Bug1040_Output2.svg.txt

---

It seems to be a more complicated bug since editiong the file changes the Processiong of other parts.

Copyright

License: CC0

Author: Mikael Häggström

Details (and Source): https://commons.wikimedia.org/wiki/File:2016_Angola_and_DR_Congo_yellow_fever_outbreak.svg

[TrySound]

convertStyleToAttrs will be disabled by default in the next release
#1365

[JoKalliauer]

@TrySound : Does "solved in master" mean, that the bug does not occur with default configuration, but might occur with different settings? Does it mean that features which are disabled by default are marginally maintained?

[TrySound]

Solved the problem with default configuration, right. I consider this plugin unsafe for now. Though removing it would be breaking change. Maybe it will come back later with more flexible architecture.

[JoKalliauer]

I make workarounds for LibrsvgBugs, librsvg might be more buggy than svgo, so convertStyleToAttrs is usefull for me, even I know it is buggy.
I prefer having buggy beta-features, even if they are not officially maintained.

Maybe keep buggy features and declare in the Docs that they are buggy and not officially maintained, and hide them e.g. in svgo --show-plugins. Maybe make a terminal-warning-output if enabled.

[TrySound]

Sure, if the problem will not be resolved I will add warning or something.

[TrySound]

https://github.com/svg/svgo/releases/tag/v2.1.0