PocketSVG icon indicating copy to clipboard operation
PocketSVG copied to clipboard

No spaces between Arc's large-arc-flag, sweep-flag and x is not handled

Open stevekuo opened this issue 3 years ago • 1 comments

It appears that removing the spaces between Arc's large-arc-flag, sweep-flag and x is permitted by popular SVG renders. I can't find this in the SVG spec but web browsers and this SVG path editor permits it.

Example: M 0 0 a 2 1 0 0 0 6 0 is

x-axis-rotation=0 large-arc-flag=0 sweep-flag=0 dx=6, dy=0

which can be shorted as M 0 0 a 2 1 0 006 0. PocketSVG complains that a does not have 7 elements

Other combinations are also permitted but not handled by PocketSVG, e.g.

M 0 0 a 2 1 0 00 6 0 is the same as M 0 0 a 2 1 0 0 0 6 0

stevekuo avatar Feb 19 '22 17:02 stevekuo

Yeah, seems like this should be handled. The grammar is here: https://www.w3.org/TR/SVG2/paths.html#PathDataBNF

fjolnir avatar Feb 26 '22 10:02 fjolnir

fixed via https://github.com/pocketsvg/PocketSVG/pull/205

arielelkin avatar Jan 09 '23 17:01 arielelkin