Missing handles after reset of rotation center

Bug #1657874 reported by theozh on 2017-01-19
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Inkscape
Low
Jabiertxof
0.92.x
Low
Jabiertxof

Bug Description

Shift+Click on rotation center should reset the rotation center to center if moved beforehand.

For example draw a rectangle...
Move rotation center:
1) click on object. Scaling handles appear
2) click 2nd time on object. Rotation center and Rotation/Skew handles appear
3) drag rotation center somewhere else
4) deselect object by clicking somewhere outside of the object
Reset rotation center:
5) repeat 1) and 2)
6) Shift+Click on Rotation center. Handles will disappear and rotation center does NOT "jump" back to the center of the object.
7) deselect object by clicking somewhere outside of the object
8) select object again. I would expect handles appear again but they do not! Just the dashed bounding box without handles.

I get back the handles only if I move the object or double click the object ending up editing the rectangle. After this the rotation center is moved back to the center.

It's observed it on Win7 /IS 0.92 system, but I also on Win10 / IS 0.92.
On Win10/Inkscape0.91 the rotation center was immediately correctly reset to the center by Shift+Click.
So it is probably a 0.92-issue...

It might be related to Bug-Report
https://bugs.launchpad.net/inkscape/+bug/1656719

Related branches

su_v (suv-lp) wrote :

Based on tests with archived builds (on OS X 10.7.5):
- not reproduced with Inkscape 0.91 r13725,
- not reproduced with lp:inkscape/0.92.x rev <= 15237,
- reproduced with lp:inkscape/0.92.x rev >= 15238,
- reproduced with Inkscape 0.92.0 r15299,
- reproduced with lp:inkscape/0.92.x rev 15322;
it seems that the reported regression was exposed with the changes in r15238:
https://bazaar.launchpad.net/~inkscape.dev/inkscape/0.92.x/changes/15238

Revision 15238: Backport trunk 15293. Add triangle knot.
https://bazaar.launchpad.net/~inkscape.dev/inkscape/0.92.x/revision/15238

Revision 15293: Add triangle knot.
https://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/revision/15293

tags: added: regression selection ui
removed: handles reset roation-center
Changed in inkscape:
status: New → Confirmed
su_v (suv-lp) wrote :

Reproduced on Ubuntu 14.04.5 LTS with
- Inkscape 0.92.0 (from stable PPA)
- lp:inkscape r15423 (local trunk build)

jazzynico (jazzynico) on 2017-01-20
Changed in inkscape:
importance: Undecided → Low
milestone: none → 0.92.1
status: Confirmed → Triaged
su_v (suv-lp) wrote :

@Jabiertxof - any chance you could take a quick whether this could be easily fixed? AFAICT based on my tests, the regression seems related to/exposed with the merge of the triangle knot branch (for mesh gradients).

su_v (suv-lp) wrote :

Actually, the bug is simply triggered by clicking on the rotation center. This happens easily with small objects and requires each time an explicit tool switch to restore the transformation handles (I encountered this many times recently when editing an Inkscape icons file with rather small objects).

Steps to reproduce:
1) launch inkscape (0.92.x)
2) draw a rect
3) click once (LMB) to show the rotation transformation handles
4) click once (LMB) on the rotation center

-> transformation handles are gone, and don't re-appear unless there's a tool switch.

jazzynico (jazzynico) on 2017-02-05
Changed in inkscape:
milestone: 0.92.1 → 0.93

The easiest way to reproduce this thing is to draw a rectangle and just click at any handle without moving mouse around. If you actually try moving object which handles you've just made disappear, you will be moving that object as if you started dragging it by clicking at point nearest to handle.

Jabiertxof (jabiertxof) on 2017-02-05
Changed in inkscape:
assignee: nobody → Jabiertxof (jabiertxof)
Jabiertxof (jabiertxof) wrote :

this patch fixes the bug

Jabiertxof (jabiertxof) wrote :

Maybe change milestone to 0.92.2?

Jabiertxof (jabiertxof) wrote :

The patch is for trunk. Apply to 0.92.x with 3 offset hunks

su_v (suv-lp) wrote :

Patch tested successfully with lp:inkscape/0.92.x r15356 on OS X 10.7.5 - thx a lot, Jabiertxof!

So far, no unintended side-effects encountered (select tool, mesh tool) during a few quick sessions - please everyone who can, test this patch as much as possible: IMvHO it would be great to get this fix accepted for the stable release branch as soon as possible.

jazzynico (jazzynico) on 2017-02-05
Changed in inkscape:
status: Triaged → In Progress
Hachmann (marenhachmann) wrote :

I've also finally tested :)

I can no longer trigger the persistent missing handles with the patch. Thank you, Jabier :)

I can still trigger the intermittently missing handles, when clicking on different objects quickly. It doesn't take a click on a handle to get this (or rather, lose this). But I've now found that this issue already exists in 0.91 (and its severity is much lower than this one's).

When clicking around a bit, working with different tools, moving and resetting rotation centers, playing with snapping etc. for a couple of minutes, I couldn't find an adverse effect of the change.

Bryce Harrington (bryce) wrote :

I always worry signal changes might have side effects but this patch looks straightforward enough. su_v says the bug is marked Low priority because an easy workaround exists, but that the user impact of this is severe enough to warrant inclusion in the point release. On the basis of the two independent testers' +1s, this is ack'd for landing on 0.92.x for inclusion in 0.92.1. Get it in quickly though.

Jabiertxof (jabiertxof) wrote :

A tweak of the patch to noyt call two times to the function on gradient drag

Jabiertxof (jabiertxof) wrote :

Fixed compiling. Thanks su_v

su_v (suv-lp) wrote :
Changed in inkscape:
status: In Progress → Fix Committed
su_v (suv-lp) wrote :

Second patch tested successfully with lp:inkscape/0.92.x on Ubuntu 14.04 and OS X 10.7.5 - AFAICT it's as stable as the first version.

Jabiertxof (jabiertxof) wrote :

Commited in 0.92.x in r.15364.
Thanks su_v for review it!

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

Duplicates of this bug

Other bug subscribers