From: Paul Bakaus <paul.bakaus@googlemail.com>
Date: Sun, 1 Jun 2008 15:56:49 +0000 (+0000)
Subject: sortable: fixed issue when no placeholder is defined, and a issue with the element... 
X-Git-Tag: 1.5.1~245
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=1c2d48f1302acee59d07c9897e280db4bf09dd0f;p=jquery-ui.git

sortable: fixed issue when no placeholder is defined, and a issue with the element shown without positioning at drag start
---

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);
 		}
 	}));