You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Not sure if this is because of the elliptical arc handling or how the path is being closed, but the current (as of 4.0 beta 8) SVG parsing produces this:
Probably something fairly simple, just needs someone to start at the path parsing a bit and find what's wrong. (And make sure the fix doesn't break other SVGs).
@benfry Is this issue still of interest?
I repro'd and played with PShapeSVG a bit and it looks like the issue is specifically when there is no delimiter between two numbers in the SVG path, e.g. a 1,1,0,0,1,.21.33. The current code seems to treat .21.33 as a single token, whereas my browser seems to render the same SVG just fine.
I think I'm able to resolve it by keeping track of whether we're already inside a decimal and using that flag to add a separator when encountering a second decimal.
Let me know if a code snippet or PR would be helpful :)
Activity
processing-bot commentedon Jul 31, 2022
Created by: benfry
This may be the same as #518
processing-bot commentedon Jan 25, 2023
Created by: bsapozhnikov
@benfry Is this issue still of interest?
I repro'd and played with PShapeSVG a bit and it looks like the issue is specifically when there is no delimiter between two numbers in the SVG path, e.g.
a 1,1,0,0,1,.21.33. The current code seems to treat.21.33as a single token, whereas my browser seems to render the same SVG just fine.I think I'm able to resolve it by keeping track of whether we're already inside a decimal and using that flag to add a separator when encountering a second decimal.
Let me know if a code snippet or PR would be helpful :)
processing-bot commentedon Jan 28, 2023
Created by: benfry
Yes, that sounds correct. Though I thought that had been fixed already in a previous PR.
But yes, it's still of interest; thank you!
processing-bot commentedon Feb 18, 2023
Created by: benfry
Integrated the fix from #659 for Processing 4.2.