Comment 5 for bug 1568644

Revision history for this message
Antonio Ospite (ospite) wrote :

I took a look and the problem is that the changes from[1] introduce a bug in ControlManagerImpl::setSelected(), see lines numbers 333 and 337 in https://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/revision/14761/src/ui/control-manager.cpp

The problem is that the previos code was changin targetSize directly while the new code sets item->ctrlResize = 2 conditionally but then does not use this value for the actual size, but the fixed _resize value.

The attached patch fixes the issue.

If the patch is OK let me know if you are going to apply it with git-am or if you prefer a merge request on gitlab.

Ciao,
   Antonio