Sprite editor select tool jumbles the selection content when moved

[ddelemeny]

Compiled from 92ced62 on ubuntu

Selecting beyond the frame of the sprite editor causes the selected content to get corrupted when moved around with the arrow keys.

Steps to reproduce :

• open the sprite editor and center on a drawn part of the spritesheet
• use the select tool and drag the selection outside of the canvas (leave some space to be able to move), release the mouse button but keep the mouse out of canvas
• move the selection around with an arrow key -> content gets corrupted

What's happening :

Keeping the mouse out of canvas prevents the release of the drag and the call to copySelection (set a breakpoint there to see it's not reached in that case). Therefore, when the selection is moved, it considers the pending selected area, but not the corresponding data.

[nesbox]

fixed here 38d4ed7

[ddelemeny]

Hmm, your fix resets the selection when you drag outside of canvas, which can be an annoying behavior when you select close to the border and you accidentally slip out.
This is why I was checking that the left mouse button wasn't pressed as a condition for drag release (albeit not on the appropriate rect I suppose).

Also, there's one corner case isn't fixed by either solution : when the user moves the selection while dragging.

Please reopen.

[ddelemeny]

@nesbox The head ddelemeny@c2ee5b9 referenced above tries to wrap it all up (based on your solution). Please have a look.

[nesbox]

yes, you are right, it resets the selection

[nesbox]

So, I reverted my previous fix and just disabled keyboard handling when you dragging a selection rectangle. It looks much simpler than other solutions 9352a37.
Thank you for your help.