Permalink
Browse files

Draggable: Ensure css is always restored after connectToSortable drag

Fixes #9675
  • Loading branch information...
mikesherov committed Aug 25, 2014
1 parent 1cc3807 commit 27ed20715f4b3e256f5279825ac551bbfcdfbe81
Showing with 8 additions and 7 deletions.
  1. +8 −7 ui/draggable.js
View
@@ -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

0 comments on commit 27ed207

Please sign in to comment.