Skip to content

Commit

Permalink
Mouse: Fixed #3552, reverted r1763: Let mouse events propagate.
Browse files Browse the repository at this point in the history
  • Loading branch information
scottgonzalez committed Jan 28, 2009
1 parent 61f39e1 commit cc3064b
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions ui/ui.core.js
Expand Up @@ -412,6 +412,9 @@ $.ui.mouse = {
},

_mouseDown: function(event) {
// don't let more than one widget handle mouseStart
if (event.originalEvent.mouseHandled) { return; }

// we may have missed mouseup (out of window)
(this._mouseStarted && this._mouseUp(event));

Expand Down Expand Up @@ -455,11 +458,7 @@ $.ui.mouse = {
// anymore, so this fix is needed
($.browser.safari || event.preventDefault());

// was changed to return true in rev 1000 because that allows the
// event to bubble - however, exactly that causes much pain for
// nested widgets, so we call stopPropagation(). Building a work-
// around for the above is much easier.
event.stopPropagation();
event.originalEvent.mouseHandled = true;
return true;
},

Expand Down

0 comments on commit cc3064b

Please sign in to comment.