jQuery UI Development

Timestamp:

09/16/08 06:02:21 (18 months ago)

Author:

paul.bakaus

Message:

draggable, resizable, sortable: proper usage of _mouseCapture, fixes interaction issues when many plugins are used on the same element (Fixes #3164)

Location:

trunk/ui

Files:

• ui.draggable.js (modified) (1 diff)
• ui.resizable.js (modified) (2 diffs)
• ui.sortable.js (modified) (1 diff)

trunk/ui/ui.draggable.js

@@ -56 +56 @@
                 
         },
-        _mouseStart: function(e) {
-                
+
+        _mouseCapture: function(e) {
+
                 var o = this.options;
                 
                 if (this.helper || o.disabled || $(e.target).is('.ui-resizable-handle'))
                         return false;
-                
+                        
                 //Quit if we're not on a valid handle
-                var handle = this.getHandle(e);
-                if (!handle)
+                this.handle = this.getHandle(e);
+                if (!this.handle)
                         return false;
+                
+                return true;
+
+        },
+
+        _mouseStart: function(e) {
+                
+                var o = this.options;
                 
                 //Create and append the visible helper

trunk/ui/ui.resizable.js

@@ -258 +258 @@
                 }
         },
-        _mouseStart: function(e) {
+        
+        _mouseCapture: function(e) {
+                
                 if(this.options.disabled) return false;
                 
@@ -266 +268 @@
                 }
                 if (!handle) return false;
+                
+                return true;
+                
+        },
+        
+        _mouseStart: function(e) {
                 
                 var o = this.options, iniPos = this.element.position(), el = this.element, 

trunk/ui/ui.sortable.js

@@ -362 +362 @@
         
         _mouseCapture: function(e, overrideHandle) {
-        
+
                 if(this.options.disabled || this.options.type == 'static') return false;