Skip to content
Permalink
Browse files

sortable: fixed issue when no placeholder is defined, and a issue wit…

…h the element shown without positioning at drag start
  • Loading branch information
Paul Bakaus
Paul Bakaus committed Jun 1, 2008
1 parent 75fdffa commit 1c2d48f1302acee59d07c9897e280db4bf09dd0f
Showing with 2 additions and 2 deletions.
  1. +2 −2 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);
}
}));

0 comments on commit 1c2d48f

Please sign in to comment.
You can’t perform that action at this time.