[Node tool] Scale/stretch/skew transforms with transform handles don't snap

Bug #590261 reported by su_v
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Low
Diederik van Lierop

Bug Description

Inkscape 0.47+devel r9486 on OS X 10.5.8, default settings

Steps to reproduce:
1) show grid
2) draw a path
3) switch to node tool, activate 'Show Transform Handles'
4) scale, stretch or skew the selection by dragging a transform handle with the mouse, snapping to the grid

Expected result:
The same as when scaling/skewing with the select tool: (node) snapping happens according to the settings in the snap controls bar.

Actual result:
Snapping is disabled when transforming nodes using the transform handles

Workaround:
None

Revision history for this message
Diederik van Lierop (mail-diedenrezi) wrote :

Probably snapping has never been implemented before for this workflow. I wonder though where in the code base the transformations are calculated; If it re-uses the selector tool's code then snapping should probably already work, so I guess some code has been duplicated.

Changed in inkscape:
assignee: nobody → Diederik van Lierop (mail-diedenrezi)
Revision history for this message
jazzynico (jazzynico) wrote :

Reproduced on Ubuntu 10.04, Inkscape r9535.

Changed in inkscape:
status: New → Confirmed
Revision history for this message
su_v (suv-lp) wrote :

> Probably snapping has never been implemented before for this workflow

As expected, this also affects dragging the rotation center of a selection of nodes -> no snapping (r9636).

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

See also related (?) request:
Bug #659599 “[node tool] Scale selections around center of rotation”:
<https://bugs.launchpad.net/inkscape/+bug/659599>

Changed in inkscape:
importance: Undecided → Low
Revision history for this message
su_v (suv-lp) wrote :

Snapping of rotation center has been filed separately:
Bug #667072 “Nodes rotating center doesn't snap on grid”:
<https://bugs.launchpad.net/inkscape/+bug/667072>

Revision history for this message
Diederik van Lierop (mail-diedenrezi) wrote :
Changed in inkscape:
status: Confirmed → In Progress
Revision history for this message
Diederik van Lierop (mail-diedenrezi) wrote :

Fully implemented as of rev. #10633

Changed in inkscape:
status: In Progress → Fix Committed
su_v (suv-lp)
Changed in inkscape:
milestone: none → 0.49
Revision history for this message
su_v (suv-lp) wrote :

@Diederik - many thanks! (much appreciated :) )

Allow me to add a question about a small related issue: any chance that bug #590260 could be revisited (show all four skew handles for horizontally or vertically aligned nodes (i.e. straight line segments) like in the select tool)?

Revision history for this message
Diederik van Lierop (mail-diedenrezi) wrote : Re: [Bug 590261] Re: [Node tool] Scale/stretch/skew transforms with transform handles don't snap

On 09/17/2011 09:52 AM, ~suv wrote:
> @Diederik - many thanks! (much appreciated :) )
>
> Allow me to add a question about a small related issue: any chance that
> bug #590260 could be revisited (show all four skew handles for
> horizontally or vertically aligned nodes (i.e. straight line segments)
> like in the select tool)?
>
@~suv: Thank you too!

I've asked Krzysztof to reconsider his opinion, which he expressed in
comment #5 of that bug report. It's his work, so it should be his call IMHO.

Bryce Harrington (bryce)
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.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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