Skip to content
Permalink
Browse files

draggable, resizable, sortable: proper usage of _mouseCapture, fixes …

…interaction issues when many plugins are used on the same element (Fixes #3164)
  • Loading branch information...
Paul Bakaus
Paul Bakaus committed Sep 16, 2008
1 parent cef66ee commit 9d8e83845cc92e2f5b1671b3bdf4a7d903844283
Showing with 24 additions and 7 deletions.
  1. +14 −5 ui/ui.draggable.js
  2. +9 −1 ui/ui.resizable.js
  3. +1 −1 ui/ui.sortable.js
@@ -55,18 +55,27 @@ $.widget("ui.draggable", $.extend({}, $.ui.mouse, {
this._mouseInit();

},
_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
this.helper = this.createHelper();

@@ -257,7 +257,9 @@ $.widget("ui.resizable", $.extend({}, $.ui.mouse, {
_destroy(wrapped);
}
},
_mouseStart: function(e) {

_mouseCapture: function(e) {

if(this.options.disabled) return false;

var handle = false;
@@ -266,6 +268,12 @@ $.widget("ui.resizable", $.extend({}, $.ui.mouse, {
}
if (!handle) return false;

return true;

},

_mouseStart: function(e) {

var o = this.options, iniPos = this.element.position(), el = this.element,
num = function(v) { return parseInt(v, 10) || 0; }, ie6 = $.browser.msie && $.browser.version < 7;
o.resizing = true;
@@ -361,7 +361,7 @@ $.widget("ui.sortable", $.extend({}, $.ui.mouse, {
},

_mouseCapture: function(e, overrideHandle) {

if(this.options.disabled || this.options.type == 'static') return false;

//We have to refresh the items data once first

0 comments on commit 9d8e838

Please sign in to comment.
You can’t perform that action at this time.