This does fix the slow rotation issue for me, but only partially. I can't seem to rotate the selector past +-45 degrees however.

This does fix the slow rotation issue for me, but only partially. I can't seem to rotate the selector past +-45 degrees however.

The limitation to +-45 degrees rotation is expected to avoid issues with the handle assignment.

Fixes the angle synchronisation for me as well.

The limitation to +-45 degrees rotation is expected to avoid issues with the handle assignment.

Do you recall what specifically happens, or how to reproduce those issues? I could not notice much when removing the 45˚ limit; only when approaching 180˚ the rotation seems to become discontinuous.
I was also wondering how the handle assignments change in comparison when "flipping" the widget by resizing, i.e. set thecenter state and drag the handle past the central point.

Sorry, I don't know more what I already mentioned in#20839 (comment).

I can confirm your expectation on theself._active_handle – when tracking it e.g. through theonselect function

defonselect(epress,erelease=None):print(tool._active_handle,tool.rotation)

a given corner handle keeps its label through any number of rotations, whereas e.g. inverting the rectangle – click theSW corner and drag it to the opposite position at (20.0, 1.6) withctrl pressed, it remainsSW through this operation, but becomesNE on the next event. So in this case it is obviously possible to update the handle labels, but I haven't found yet if the same could be done forabs(self.rotation) > 45 (which would also need investigation if the angle should then be reset toself.rotation % 45...).
The main practical problem I have found so far is that the resizing becomes more and more unintuitive through the trochoidal reflex motion of the centre handle, though personally I do not find that problematic up to 90˚ angles.

@dhomeier, is it worth discussing this in a dedicated issue or PR? If not, the discussion will get lost as this PR is not about increasing the rotation limits.

I saw that@dstansby mentioned#21921 as part of an effort to extend the rotation range, but have not found an associated issue for that.

Yes, lets move to another issue - I'll open one.

I now have a working version ofRectangleSelector that removes the 45 degree limitation, and also fixes scaling using an edge handle when rotated. It needs a bit more work to iron out (and add some more!) tests, but I think it's worth holding off on this for now since it would be redundant with my newer version.

I now have a working version ofRectangleSelector that removes the 45 degree limitation, and also fixes scaling using an edge handle when rotated. It needs a bit more work to iron out (and add some more!) tests, but I think it's worth holding off on this for now since it would be redundant with my newer version.

Yes, makes sense, I am happy to give an early review, even if this is not finish yet! :)

I guess I'm not sure about merging this? Feel free to do so if you think it doesn't need a Whats new entry, and it is consistent with the current state of the docs...

It makes sense to put this on hold for now, because it will not be necessary after#21945 is merged and#21945 is already in good shape.

Pushing to 3.8 as the last comment suggests waiting for#21945