From 1c2d48f1302acee59d07c9897e280db4bf09dd0f Mon Sep 17 00:00:00 2001 From: Paul Bakaus Date: Sun, 1 Jun 2008 15:56:49 +0000 Subject: [PATCH] sortable: fixed issue when no placeholder is defined, and a issue with the element shown without positioning at drag start --- ui/source/ui.sortable.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ui/source/ui.sortable.js b/ui/source/ui.sortable.js index 17287bb86..d0eca57b2 100644 --- a/ui/source/ui.sortable.js +++ b/ui/source/ui.sortable.js @@ -396,7 +396,7 @@ this.dragging = true; - //this.mouseDrag(e); //Execute the drag once - this causes the helper not to be visible before getting its correct position + this.mouseDrag(e); //Execute the drag once - this causes the helper not to be visible before getting its correct position return true; @@ -562,7 +562,7 @@ rearrange: function(e, i, a) { a ? a.append(this.currentItem) : i.item[this.direction == 'down' ? 'before' : 'after'](this.currentItem); this.refreshPositions(true); //Precompute after each DOM insertion, NOT on mousemove - this.options.placeholder.update.call(this.element, this.currentItem, this.placeholder); + if(this.options.placeholder) this.options.placeholder.update.call(this.element, this.currentItem, this.placeholder); } }));