From fd8905bcd217ad617f3f28e2a8125697b15d2d2f Mon Sep 17 00:00:00 2001 From: Paul Bakaus Date: Sat, 7 Jun 2008 14:01:33 +0000 Subject: sortable: fixed update calls (fixes #2968) --- ui/source/ui.sortable.js | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/ui/source/ui.sortable.js b/ui/source/ui.sortable.js index 2f7763e90..a433805e8 100644 --- a/ui/source/ui.sortable.js +++ b/ui/source/ui.sortable.js @@ -277,7 +277,7 @@ $.widget("ui.sortable", $.extend($.ui.mouse, { }; }, mouseStart: function(e, overrideHandle, noActivation) { - + var o = this.options; this.currentContainer = this; @@ -521,9 +521,10 @@ $.widget("ui.sortable", $.extend($.ui.mouse, { }, clear: function(e, noPropagation) { - - if(this.domPosition != this.currentItem.prev()[0]) this.propagate("update", e, null, noPropagation); //Trigger update callback if the DOM position has changed + + if(this.domPosition != this.currentItem.prev().not(".ui-sortable-helper")[0]) this.propagate("update", e, null, noPropagation); //Trigger update callback if the DOM position has changed if(!contains(this.element[0], this.currentItem[0])) { //Node was moved out of the current element + if(this.domPosition == this.currentItem.prev().not(".ui-sortable-helper")[0]) this.propagate("update", e, null, noPropagation); //Trigger update callback, if it wasn't triggered before, because that is definitely an update this.propagate("remove", e, null, noPropagation); for (var i = this.containers.length - 1; i >= 0; i--){ if(contains(this.containers[i].element[0], this.currentItem[0])) { -- cgit v1.2.3