Opened 2 years ago

Closed 19 months ago

Last modified 19 months ago

#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 2 years ago by Michael Lelli

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.

comment:2 Changed 2 years ago by Scott González

Component: ui.draggableui.mouse
Status: newopen
Summary: draggable stops mouseup eventsMouse 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 20 months ago by Scott González

Keywords: help-wanted added

comment:4 Changed 19 months ago by Hannah Methvin

Resolution: fixed
Status: openclosed

Mouse: Don't stop propagation of mouseup

Fixes #10818

Changeset: a1d69208bad175a27c7b50c27fdc10001563cd4d

comment:5 Changed 19 months ago by Scott González

Milestone: none1.12.0

comment:6 Changed 19 months ago by Scott González

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

Note: See TracTickets for help on using tickets.