From: Mike Sherov <mike.sherov@gmail.com>
Date: Mon, 25 Aug 2014 01:08:02 +0000 (-0400)
Subject: Draggable: Ensure css is always restored after connectToSortable drag
X-Git-Tag: 1.11.2~29
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=27ed20715f4b3e256f5279825ac551bbfcdfbe81;p=jquery-ui.git

Draggable: Ensure css is always restored after connectToSortable drag

Fixes #9675
---

diff --git a/ui/draggable.js b/ui/draggable.js
index d290ea539..ba8eca42b 100644
--- a/ui/draggable.js
+++ b/ui/draggable.js
@@ -744,19 +744,20 @@ $.ui.plugin.add("draggable", "connectToSortable", {
 				if (this.shouldRevert) {
 					this.instance.options.revert = this.shouldRevert;
 				}
+				// Use _storedCSS To restore properties in the sortable,
+				// as this also handles revert (#9675) since the draggable
+				// may have modified them in unexpected ways (#8809)
+				this.instance._storedCSS = {
+					position: this.instance.placeholder.css( "position" ),
+					top: this.instance.placeholder.css( "top" ),
+					left: this.instance.placeholder.css( "left" )
+				};
 
 				//Trigger the stop of the sortable
 				this.instance._mouseStop(event);
 
 				this.instance.options.helper = this.instance.options._helper;
 
-				// restore properties in the sortable, since the draggable may have
-				// modified them in unexpected ways (#8809)
-				this.instance.currentItem.css({
-					position: this.instance.placeholder.css( "position" ),
-					top: "",
-					left: ""
-				});
 			} else {
 				// Prevent this Sortable from removing the helper.
 				// However, don't set the draggable to remove the helper