SVG closepath-Command

[hefee]

Hello,

I read the W3C SVG 1.1 Specification from 14 January 2003 [http://www.w3.org/TR/2003/REC-SVG11-20030114/paths.html#PathDataClosePathCommand] and think I found a bug in the svg implementation. If an path is closed to the beginning point an close-path command "Z" is added. Inkscape creates another line-to command "L" to the beginning point is added:

Inkscape makes:
M 0,0 L 1,0 L 0,1 L 0,0 Z

from svg standard i think it should be:
M 0,0 L 1,0 L 0,1 Z

Dear
hefee

[su_v (suv-lp)]

Closing as 'Fix released' based on: Release notes 0.47 - No more explicit closing line segment
<http://wiki.inkscape.org/wiki/index.php/Release_notes/0.47#No_more_explicit_closing_line_segment>

«Inkscape used to always explicitly write the closing line segment to SVG for closed paths. For example: "M 0,0 L 1,0 L 1,1 L 0,1 L 0,0 z". With the improved SVG path data writing, Inkscape no longer generates this closing line segment, resulting in "M 0,0 L 1,0 L 1,1 L 0,1 z".»

[su_v (suv-lp)]

Sorry, I prematurely closed this bug as fixed without proper testing. The optimized path data writing doesn't work at the moment…
<http://wiki.inkscape.org/wiki/index.php?title=Release_notes%2F0.47&diff=54139&oldid=54137>

[Johan Engelen (johanengelen)]

It is a bit strange.. it *used* to work (which is why I added it to the release notes), however, I don't think it has high priority. Inkscape 0.46 also has this behavior, so maybe we can fix it / optimize for 0.48. I'm not too sure about the differences between "M 0,0 L 1,0 L 0,1 L 0,0 Z" and "M 0,0 L 1,0 L 0,1 Z". Perhaps mid-markers will show differently for both paths...

[Johan Engelen (johanengelen)]

Ok I just checked it again.
Inkscape still generates new paths like "M 0,0 L 1,0 L 0,1 L 0,0 Z". However, if you start with "M 0,0 L 1,0 L 0,1 Z", it is kept that way (without explicit closing segment to the beginning of the path). It is kept in this form when doing transforms and maybe when doing other things. However, when you use the node tool, Inkscape will add the explicit closing segment (like "M 0,0 L 1,0 L 0,1 L 0,0 Z"). This is probably because the nodetool converts 2geom path to its own path representation and then back to 2geom. Maybe the new nodetool by Krzystof will not have this problem.

[hefee (hefee)]

If I understand the w3c correct, the Z command draws a straight line from the actual Point to the starting Point.

Here the important sentence from w3c: "The "closepath" (Z or z) ends the current subpath and causes an automatic straight line to be drawn from the current point to the initial point of the current subpath."

So M 0,0 L 1,0 L 0,1 L 0,0 Z is a polygon with 5 lines ( 0,0-1,0; 1,0-0,1; 0,1:0,0; 0,0:0,0) and 4 points ( 0,0; 0,1; 1,0; 0,0) ( two times the point 0,0).

and M 0,0 L 1,0 L 0,1 Z has 4 lines (0,0-1,0; 1,0-0,1; 0,1:0,0) and 3 points (0,0; 0,1; 1,0)

[su_v (suv-lp)]

@Johan - yes, I "re-visited" the release notes and this bug after testing bug #403445 “polygons with mid markers show 1 too many [Regression]” with a current build.

[Krzysztof Kosinski (tweenk)]

This is fixed in the new node tool. There is no extra linear segment appended if the last segment is linear.