Rotation handlers not working for lines

Bug #731141 reported by LucaDC
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Medium
Diederik van Lierop

Bug Description

I was playing with last Diederik's fix for selecting the rotation center of zero-dimension objects (thanks a lot fot that!) and I found out a problem with rotation handlers:
 - draw a horizontal line, press F1 and try all four rotation handlers: some don't work;
 - draw an oblique line, press F1 and try all four rotation handlers: some make the line become horizontal and that's all you can do.
Regards.

su_v (suv-lp)
tags: added: selection transformations
Revision history for this message
su_v (suv-lp) wrote :

Not reproduced with r10082 on OS X 10.5.8 (i386), tested with my normal settings and with new, default preferences.

Could you attach your preferences.xml file, and a sample SVG file, so that we can test with the same settings?

Revision history for this message
LucaDC (lucadc) wrote :

Here they are.

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

Reproduced with Inkscape 0.48+devel r10082 on OS X 10.5.8 (i386):
New, default settings with additionally 'Snap from and to midpoints of line segments' activated.

No longer reproduced when changing the snapping preferences to
[x] Only snap the node closest to the pointer

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

@Diederik - could you take a look at this issue? AFAICT it's not related to your latest commit r10081(for me it also occurs e.g. with r10078 or r10060), but it's also not in the 0.48.x branch (not reproduced with Inkscape 0.48.1).

Changed in inkscape:
assignee: nobody → Diederik van Lierop (mail-diedenrezi)
importance: Undecided → Medium
status: New → Triaged
tags: added: snapping
removed: selection
Revision history for this message
LucaDC (lucadc) wrote :

You made me remember I recently had to enable the "Only snap the node closest to the pointer" option to avoid some snapping issues I had no time to figure out precisely enough to submit a bug report. I just remember some snapping happening also with its option disabled (maybe path intersection?) and node snapping failing for the first or last node of a path (but not always).
If they come out again I'll add them here with more details.
Regards.

Revision history for this message
LucaDC (lucadc) wrote :

"Only snap the node closest to the pointer" is a nice option, but then how can you snap an empty circle's center that you must pick on its border to move?
Try with the attachment: did I set some snapping option wrong?
Thanks

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

> how can you snap an empty circle's center

I normally change its fill type to solid before grabbing it at the center, and revert the fill to 'None' after the move (just two additional clicks, nothing else gets modified).

Revision history for this message
LucaDC (lucadc) wrote :

Any additional click is too much if there is an option requiring no clicks at all that would do the job if only it worked correctly.

Anyway, did you try my last attachment?
There's a bug in it, to me: quadrant points should not snap...

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

> There's a bug in it, to me: quadrant points should not snap...

IMHO not a bug: the snap settings are
'Snap nodes and handles'
'Snap to cusp nodes'
'Snap from and to centers of objects'

Snapping the closest node to a cusp corner node is correct: you cannot specify which node types snap, only *to* which node type they snap.

Revision history for this message
LucaDC (lucadc) wrote :

~Suv, you're right: that's what the selected options are supposed to do, indeed.
Actually there's no way to ask Inkscape to snap only objects' (rotation) centers. Shouldn't those two options fall outside the "Snap node or handles" group, as centers are neither nodes nor handles? This has deceived me.

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

There's indeed a problem with those horizontal lines, but only reproducible with "Only snap the node closest to the pointer" turned off and with the weightfactor set to 0.5. I will look into this

> Actually there's no way to ask Inkscape to snap only objects' (rotation) centers
Correct; it would be impossible to provide options for controlling each combination of snap source and snap target individually. It would probably require a 30x30 matrix of radio buttons ;-). Therefore some choices had to be made, and as a consequence you cannot distinguish between all of the various snap sources. So you will have to turn on "snap to/from rotation center", and turn on "snap nodes", which will include all smooth and cusp nodes, all special points such as corners of a (rounded) rectangle and center of arcs, etc.

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

That bug should have been fixed as of rev. #10093.

Revision history for this message
LucaDC (lucadc) wrote :

The bug seems fixed for me too. Thanks.

>it would be impossible to provide options for
>controlling each combination of snap source and snap
>target individually. It would probably require a 30x30
>matrix of radio buttons ;-).
Agreed. But in this case the buttons are already present :)
I was just proposing to "logically" move them outside the "node" group so, for example, we'll be able to snap a center to a bounding box corner or to a grid, having all else disabled (nodes in particular, which tend to be so many all around...). Would this be a problem or difficoult to realize?

Actually, I can't fully appreciate the distinction between snapping sources and targets: I don't find it really helpful while I tend to assume that pushing a single "snap nodes" button adds all "nodes" to the "snapping pool" (made of all possible snapping sources and targets). Why snapping nodes but not snapping to nodes? And why not the opposite? I understand that having a distincion adds freedom but as someone said, this could end up in a n^2 matrix... ;)

Regards

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

Setting status to 'Fix Released' since the bug was not reproduced in the stable branch 0.48.x.

Changed in inkscape:
status: Triaged → 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.