#10818 closed bug (fixed)
Mouse stops propagation of events to window
Reported by: | Michael Lelli | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 1.12.0 |
Component: | ui.mouse | Version: | 1.11.2 |
Keywords: | help-wanted | Cc: | |
Blocked by: | Blocking: |
Description
When assigning draggable to an element, any mouseup events on parent elements will not be fired. mousedown events still fire like normal so this seems like it's incorrect behavior.
Testcase: http://jsfiddle.net/wru1zdzs/
Change History (6)
comment:1 Changed 8 years ago by
comment:2 Changed 8 years ago by
Component: | ui.draggable → ui.mouse |
---|---|
Status: | new → open |
Summary: | draggable stops mouseup events → Mouse stops propagation of events to window |
This is because we're returning booleans in our delegated handlers instead of just preventing the default action. The reason nobody has reported this before is because binding to mouse events on the window is pretty uncommon. Just bind to the document and this will work for you right now.
comment:3 Changed 7 years ago by
Keywords: | help-wanted added |
---|
comment:4 Changed 7 years ago by
Resolution: | → fixed |
---|---|
Status: | open → closed |
Mouse: Don't stop propagation of mouseup
Fixes #10818
Changeset: a1d69208bad175a27c7b50c27fdc10001563cd4d
comment:5 Changed 7 years ago by
Milestone: | none → 1.12.0 |
---|
comment:6 Changed 7 years ago by
Draggable: Skip window bubbling test in IE 8
IE 8 implements DOM Level 2 Events which only has events propagate up to the document, not the window.
Ref #10818 Ref gh-1621 Closes gh-1627
Changeset: 33398fa4f1ee33473cc6c190b2b1d38c11b8c38b
I forgot to mention, the mouseup events still fire if you're not dragging the box around, but don't fire when you do drag it.