Permalink
Browse files

Revert "Mouse: Changed mouseHandled check to a local var rather than …

…using originalEvent. Fixed #4333 - Nested draggables problem in IE"

This reverts commit 7eda94a.
  • Loading branch information...
1 parent b796cc5 commit bd48ddfa94ff673cc5dfef749fe0565f1bc02d0f @scottgonzalez scottgonzalez committed Aug 1, 2011
Showing with 4 additions and 8 deletions.
  1. +4 −8 ui/jquery.ui.mouse.js
View
@@ -12,11 +12,6 @@
*/
(function( $, undefined ) {
-var mouseHandled = false;
-$(document).mousedown(function(e) {
- mouseHandled = false;
-});
-
$.widget("ui.mouse", {
options: {
cancel: ':input,option',
@@ -49,7 +44,9 @@ $.widget("ui.mouse", {
_mouseDown: function(event) {
// don't let more than one widget handle mouseStart
- if(mouseHandled) {return};
+ // TODO: figure out why we have to use originalEvent
+ event.originalEvent = event.originalEvent || {};
+ if (event.originalEvent.mouseHandled) { return; }
// we may have missed mouseup (out of window)
(this._mouseStarted && this._mouseUp(event));
@@ -95,8 +92,7 @@ $.widget("ui.mouse", {
.bind('mouseup.'+this.widgetName, this._mouseUpDelegate);
event.preventDefault();
-
- mouseHandled = true;
+ event.originalEvent.mouseHandled = true;
return true;
},

0 comments on commit bd48ddf

Please sign in to comment.