#10818 closed bug (fixed)

Opened January 23, 2015 05:14PM UTC

Closed October 18, 2015 06:23PM UTC

Last modified October 27, 2015 03:12PM UTC

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/

Attachments (0)
Change History (6)

Changed January 23, 2015 05:17PM UTC by Michael Lelli comment:1

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.

Changed January 23, 2015 05:31PM UTC by scottgonzalez comment:2

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.

Changed October 13, 2015 06:53PM UTC by scottgonzalez comment:3

keywords: → help-wanted

Changed October 18, 2015 06:23PM UTC by Hannah Methvin comment:4

resolution: → fixed
status: open → closed

Mouse: Don't stop propagation of mouseup

Fixes #10818

Changeset: a1d69208bad175a27c7b50c27fdc10001563cd4d

Changed October 18, 2015 06:25PM UTC by scottgonzalez comment:5

milestone: none → 1.12.0

Changed October 27, 2015 03:12PM UTC by Scott González comment:6

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