Don't drag draggables on scroll events#29842
Conversation
|
Why do we need a custom picker / why does the regular picker fire on scroll events? Should it be fixed generally on the regular picker? |
|
A scroll event comes in looking just like a button press (but with a higher number). It does not seem unreasonable to allow these to trigger pick events in general but to disable them for draggable things specifically. |
|
I don't think I understand this: "Pick" is logically selecting an element by clicking on it. No matter the technical implementation, why should scrolling ever be interpreted as pick? In what use case would that make sense? |
|
|
timhoffm
left a comment
timhoffm
left a comment
There was a problem hiding this comment.
Anybody can merge after adding the suggested documentation.
|
|
||
| @staticmethod | ||
| def _picker(artist, mouseevent): | ||
| return (artist.contains(mouseevent) and mouseevent.name != "scroll_event"), {} |
There was a problem hiding this comment.
| return (artist.contains(mouseevent) and mouseevent.name != "scroll_event"), {} | |
| """ | |
| A custom picker to prevent dragging on scroll events. | |
| Dragging is initialized through a pick events, which is typically a | |
| mouse click. However, mouse scrolling also fires pick events and | |
| we need to filter them out so that scrolling does not initalize | |
| dragging. | |
| """ | |
| return (artist.contains(mouseevent) and mouseevent.name != "scroll_event"), {} |
There was a problem hiding this comment.
👍 to adding a comment - I thought this was a bit verbose for a private helper function, so I added a one-liner instead - what do you think?
3 participants
PR summary
Fixes #29142; replaces #29270. The solution here is to set a custom picker on
DraggableBasethat doesn't fire if the event is a scroll event.PR checklist