From bd48ddfa94ff673cc5dfef749fe0565f1bc02d0f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Scott=20Gonz=C3=A1lez?= Date: Mon, 1 Aug 2011 14:17:38 -0400 Subject: [PATCH] Revert "Mouse: Changed mouseHandled check to a local var rather than using originalEvent. Fixed #4333 - Nested draggables problem in IE" This reverts commit 7eda94a8c916760de4093df85f58cc694838b52b. --- ui/jquery.ui.mouse.js | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/ui/jquery.ui.mouse.js b/ui/jquery.ui.mouse.js index 2fb1389f2b8..7e38c69c444 100644 --- a/ui/jquery.ui.mouse.js +++ b/ui/jquery.ui.mouse.js @@ -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; },