invalid attribute: fill should not be "undefined"

[chrisgarrity]

Some sprites' svgs get exported with fill="undefined". This is not a valid attribute and causes errors when the svg is imported to other systems (Android).

When this is fixed file an issue on scratch-android to remove the work-around.

[BryceLTaylor]

This is related to these issues:
#31
#32

There are workaround for Android that strip each of these properties out of svgs that are a processing burden. According to @chrisgarrity, "There are now 4 work-arounds that require reading every svg file 4 times and making substitutions as needed."

[adroitwhiz]

Any idea which sprites are being saved this way?

[adroitwhiz]

It looks like some Scratch 2.0-generated assets (e.g. the dove) in the asset library have fill="undefined", but when they pass through Paper, it (correctly) coerces that to fill="#000000".

If you change any part of the costume in the paint editor, the exported SVG will be fixed, but if you export it directly without changing anything, it'll use the Scratch 2.0 SVG structure.

The solution may be to go over the asset library with some sort of SVG validator to fix these invalid SVGs generated by 2.0.