Opened 5 years ago

Last modified 2 years ago

#18377 reopened defect

dojox/mdnd/Moveable does not fire mouseup in IE10/11 when focus is changed

Reported by: Nick Nisi Owned by: bill
Priority: undecided Milestone: 1.14
Component: Dojox Version: 1.10.2
Keywords: Cc:
Blocked By: Blocking:

Description

When clicking on a widget inside of a dojox/mdnd/Moveable widget that changes the focus to another element, such as an input field, the mouseup event is not fired, causing the Moveable widget to be dragged as the mouse is moved. Looking at #17709 seems to provide a possible explanation, and looking at the dojo/dnd/Moveable code, it looks like this could be fixed by instead listening for dojo/touch events.

Example: http://jsfiddle.net/nicknisi/m7dodbmz/

Attachments (1)

mdnd.html (1.8 KB) - added by bill 5 years ago.
test case modified to run against master

Download all attachments as: .zip

Change History (7)

comment:1 Changed 5 years ago by Nick Nisi

comment:2 Changed 5 years ago by bill

Milestone: tbd1.11
Owner: changed from Adam Peller to bill
Status: newassigned

Looks good, thanks for the patch. I had to modify your test case a tiny bit to run against master, since there's no longer a dojox.mdnd.areaManager global variable. I'll attach it to this ticket.

Changed 5 years ago by bill

Attachment: mdnd.html added

test case modified to run against master

comment:3 Changed 5 years ago by Bill Keese <bill@…>

Resolution: fixed
Status: assignedclosed

In 40b9aadaaf6eb2a5688fa5b6609c28fe9181ea57/dojox:

Error: Processor CommitTicketReference failed
Unsupported version control system "git": Can't find an appropriate component, maybe the corresponding plugin was not enabled? 

comment:4 Changed 3 years ago by mssskhalsa

Are there plans to backport this? I'm only seeing it in master.

comment:5 Changed 2 years ago by dylan

Milestone: 1.111.12.3
Resolution: fixed
Status: closedreopened

Re-opening as this fix should have been backported if possible.

comment:6 Changed 2 years ago by dylan

Milestone: 1.12.31.14
Note: See TracTickets for help on using tickets.