Opened 2 years ago

Closed 15 months ago

Last modified 15 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 scottgonzalez

  • Component changed from ui.draggable to ui.mouse
  • Status changed from new to open
  • Summary changed from draggable stops mouseup events to 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 15 months ago by scottgonzalez

  • Keywords help-wanted added

comment:4 Changed 15 months ago by Hannah Methvin

  • Resolution set to fixed
  • Status changed from open to closed

Mouse: Don't stop propagation of mouseup

Fixes #10818

Changeset: a1d69208bad175a27c7b50c27fdc10001563cd4d

comment:5 Changed 15 months ago by scottgonzalez

  • Milestone changed from none to 1.12.0

comment:6 Changed 15 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.