BSpline Path Effect: 'Change only selected nodes' first edits previously selected node

Bug #1630821 reported by Hachmann on 2016-10-06
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Undecided
Jabiertxof

Bug Description

When using the BSpline LPE, and checking 'Change only selected nodes', the 'make cusp' and 'change weight' buttons are - when used for the first time - applied to the previously selected node in the path, not to the currently selected one.

1. Draw BSpline path.
2. In LPE dialog, check 'Change only selected nodes'
3. Select a node
4. Change its weight, so you can see the bug
5
. Select another node.
6. Click on 'default weight' or 'make cusp'
7. Result: the previously selected node with the non-default weight is changed

Inkscape trunk ppa version of Oct. 4th 2016.

su_v (suv-lp) wrote :

Reproduced with lp:inkscape/0.92.x r15094 (on OS X 10.7.5).

Based on tests with recent archived builds:
- not reproduced with lp:inkscape/0.92.x rev <= 15051
- reproduced with lp:inkscape/0.92.x rev >= 15053
this regression possibly has been exposed with the changes from:

Revision 15053: Allow update helper paths from LPE widget interaction
https://bazaar.launchpad.net/~inkscape.dev/inkscape/0.92.x/revision/15053

The same changes had been committed earlier to trunk lp:inkscape in revision 15086.

tags: added: livepatheffects ui
Changed in inkscape:
status: New → Confirmed
Hachmann (marenhachmann) wrote :

Thanks for testing!

Jabiertxof (jabiertxof) on 2016-10-06
Changed in inkscape:
assignee: nobody → Jabiertxof (jabiertxof)
Jabiertxof (jabiertxof) wrote :

I fix the bug. I commit it when finish compiling.

Jabiertxof (jabiertxof) wrote :

Fixes on trunk r.15149
And in 0.92.x in 15095

Changed in inkscape:
status: Confirmed → Fix Released
Jabiertxof (jabiertxof) wrote :

Thanks for fill the bug and tests.

Hachmann (marenhachmann) wrote :

Ah, that was quick, Jabier! Thx, too!

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers