popup fails to close when dragging Moveable

[Devang Negandhi]

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,

1. Open the select box on the floating pane titled "A floating pane 2"
2. Now go and click on the title of "A floating pane 1"
3. You can see that the dropdown from the select of "A floating pane 2" show on top of "A floating pane 1"
4. 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

[bill]

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?

[bill]

OK, the evt.stopPropagation() is from dojo/dnd/Moveable.js, in the onMouseDown() method. There are a number of tickets against the DnD code to stop calling evt.stopPropagation(): #16157, #17606, #17110... and now this one.

[bill]

copy of test, to run locally

[Eugene Lazutkin]

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.

[bill]

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.

[dylan]

Revisit for 1.12, but probably need to close as it looks like we cannot fix until 2.0.

[dylan]

Ticket planning... move current 1.13 tickets out to 1.15 to make it easier to move tickets into the 1.13 milestone.