From 2e0f967680447e93e04eea2764f53ea3977b4f5f Mon Sep 17 00:00:00 2001 From: Paul Bakaus Date: Tue, 10 Feb 2009 10:07:25 +0000 Subject: [PATCH] sortable: if current item was removed by user, it was reappended by the clearing logic (fixes #4088) --- ui/ui.sortable.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ui/ui.sortable.js b/ui/ui.sortable.js index 56c613d04..e3a3f3d3d 100644 --- a/ui/ui.sortable.js +++ b/ui/ui.sortable.js @@ -901,8 +901,9 @@ $.widget("ui.sortable", $.extend({}, $.ui.mouse, { // everything else normalized again var delayedTriggers = [], self = this; - //We first have to update the dom position of the actual currentItem - if(!this._noFinalSort) this.placeholder.before(this.currentItem); + // We first have to update the dom position of the actual currentItem + // Note: don't do it if the current item is already removed (by a user), or it gets reappended (see #4088) + if(!this._noFinalSort && this.currentItem[0].parentNode) this.placeholder.before(this.currentItem); this._noFinalSort = null; if(this.helper[0] == this.currentItem[0]) { -- 2.39.5