Opened 11 years ago
Closed 5 years ago
#10732 closed defect (patchwelcome)
[patch][cla][needs amd conversion] dojox.gfx.Moveable used with delay can get into bad state
Reported by: | Owned by: | Eugene Lazutkin | |
---|---|---|---|
Priority: | high | Milestone: | 1.13 |
Component: | DojoX GFX | Version: | 1.4.0 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
Steps:
- Use dojox.gfx.Moveable on a shape and have a delay (a larger delay will cause this issue to be more prominant).
- Drag quickly on the shape to the outside such that the mouse move interval exceeds the drag and then mouse-up outside of the shape.
- move mouse back over the shape...
Result:
- If you are handling the onFirstMove override, then you may be creating feedback while your mouse button is up. From the user standpoint they have released the mouse and the gesture should be over.
Expected:
- Probably the move should be terminated when the mouse leaves the shape because the internal exceeded the delay.
I have attached a file with suggested changes in dojox.gfx.Moveable. Basically a handler for "onMouseLeave" has been added that checks if the move hasn't been started and calls this.onMouseUp to terminate the move.
Attachments (2)
Change History (5)
Changed 11 years ago by
Attachment: | Moveable.2.js added |
---|
Changed 11 years ago by
Attachment: | Moveable.3.js added |
---|
small change to centralize setting of _delayInEffect to false
comment:1 Changed 11 years ago by
Component: | General → DojoX GFX |
---|---|
Owner: | changed from anonymous to Eugene Lazutkin |
comment:2 Changed 11 years ago by
Milestone: | tbd → future |
---|---|
Status: | new → assigned |
comment:3 Changed 5 years ago by
Milestone: | future → 1.12 |
---|---|
Resolution: | → patchwelcome |
Status: | assigned → closed |
Summary: | dojox.gfx.Moveable used with delay can get into bad state → [patch][cla][needs amd conversion] dojox.gfx.Moveable used with delay can get into bad state |
This one has a very old patch, sorry that it was never reviewed 6 years ago. If someone wants to update it, we'll consider it for 1.12.
Note: See
TracTickets for help on using
tickets.
Here is a suggested fix for the issue described in Moveable.js