Opened 7 years ago
Last modified 4 years ago
#17802 new defect
popup fails to close when dragging Moveable
Reported by: | Devang Negandhi | Owned by: | Eugene Lazutkin |
---|---|---|---|
Priority: | undecided | Milestone: | 1.15 |
Component: | DnD | Version: | 1.9.3 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
If I open a Select dropdown and then go click on the title of a floating pane, the dropdown does not close. This causes a problem when we have 2 floating panes. If I have the dropdown open in one floating pane and I move a second floating pane over it, the popup from the first floating pane now shows on top of the new floating pane
Take a look at the following jsfiddle: http://jsfiddle.net/K9zk6/1/
In the above jsfiddle,
- Open the select box on the floating pane titled "A floating pane 2"
- Now go and click on the title of "A floating pane 1"
- You can see that the dropdown from the select of "A floating pane 2" show on top of "A floating pane 1"
- Even if you move the floating pane 1 around, the dropdown is still visible
I am also attaching a screenshot of the issue along with this ticket
Attachments (2)
Change History (8)
Changed 7 years ago by
Attachment: | FloatingPaneBug.png added |
---|
comment:1 Changed 7 years ago by
Component: | General → DojoX Layout |
---|---|
Summary: | dijit.form.Select popup fails to close → dijit.form.Select popup fails to close when clicking title of FloatingPane |
comment:2 Changed 7 years ago by
Component: | DojoX Layout → DnD |
---|---|
Owner: | set to Eugene Lazutkin |
Summary: | dijit.form.Select popup fails to close when clicking title of FloatingPane → popup fails to close when dragging Moveable |
comment:3 Changed 7 years ago by
My understanding is that the extensive analysis was done, and removing a call to stopPropagation() cures all related problems. Is it correct? If so, let's remove it, and close all those tickets.
comment:4 Changed 7 years ago by
No, it's not exhaustively tested... actually removing the stopPropagation() call breaks the nested moveable in test_moveable.html. I.e. moving the nested yellow div drags its parent div along with it.
I'm not sure what the right solution to this problem is. When we drop support for older browsers, the dijit popup code can listen for mousedown in the capturing phase rather than the bubbling phase, making it immune to stopPropagation() calls. But can't do that for Dojo V1.
comment:5 Changed 5 years ago by
Milestone: | tbd → 1.12 |
---|
Revisit for 1.12, but probably need to close as it looks like we cannot fix until 2.0.
comment:6 Changed 4 years ago by
Milestone: | 1.13 → 1.15 |
---|
Ticket planning... move current 1.13 tickets out to 1.15 to make it easier to move tickets into the 1.13 milestone.
Notice that clicking other parts of the screen closes the dropdown... so it's probably FloatingPane? (unnecessarily) calling evt.preventDefault() (or indirectly calling it via dojo.stopEvent() etc.), preventing dijit's focus manager from seeing the mousedown event.
PS: or possibly the evt.preventDefault() problem is in dojo/dnd/Moveable?