Path shape not maintained when adding a node

Bug #1367443 reported by Giacomo Ciani
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Medium
Alvin Penner

Bug Description

When you add a point to an existing path, the expectation is that the shape of the path is "exactly" maintained. From a geometrical point of view, I suppose this is always possible (i.e. there always exist a solution in which the additional node is added but do not change the shape of the path, rounding errors and similar excepted).

Indeed, Inkscape usually fulfills the expectations. However, I just run into an example in which this does not work. In this case, the addition substantially modifies the original path, even causing one of the other nodes to move around.

To reproduce:
- open the attached file
- you will see a short path (e nodes) resembling a wave (they are actually two identical one on top of each other, so one can serve as a reference); there is also a vertical grey line, also to serve as a reference.
- select the "edit path" tool and double click on the "wave" somewhere in the vicinity of the vertical line.
- this will add a node in the position in which you double clicked, but will also noticeably modify the shape of the wave, and cause the middle node to move from its original position

Is this a bug in the algorithm the calculates the new path with the added node?

I'm using Inkscape 0.48.5 r10040.

Tags: node-editing
Revision history for this message
Giacomo Ciani (jackseriuos79) wrote :
information type: Private Security → Public
Revision history for this message
Giacomo Ciani (jackseriuos79) wrote :

Sorry, I marked this as "security vulnerability" by mistake.

Revision history for this message
su_v (suv-lp) wrote :

Reproduced with Inkscape 0.47, 0.48.5 r10040 and 0.91pre2 r13549 on OS X 10.7.5.

tags: added: node-editing
Changed in inkscape:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Alvin Penner (apenner) wrote :

fix committed to rev 13800

Changed in inkscape:
status: Confirmed → Fix Committed
assignee: nobody → Alvin Penner (apenner)
su_v (suv-lp)
Changed in inkscape:
milestone: none → 0.92
Revision history for this message
ScislaC (scislac) wrote :

Fix backported in 0.91.x r13707.

Changed in inkscape:
milestone: 0.92 → 0.91
su_v (suv-lp)
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.