mirror of
https://github.com/jquery/jquery-ui.git
synced 2024-11-21 11:04:24 +00:00
sortable: fixed issue when no placeholder is defined, and a issue with the element shown without positioning at drag start
This commit is contained in:
parent
75fdffacb0
commit
1c2d48f130
@ -396,7 +396,7 @@
|
|||||||
|
|
||||||
this.dragging = true;
|
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;
|
return true;
|
||||||
|
|
||||||
|
|
||||||
@ -562,7 +562,7 @@
|
|||||||
rearrange: function(e, i, a) {
|
rearrange: function(e, i, a) {
|
||||||
a ? a.append(this.currentItem) : i.item[this.direction == 'down' ? 'before' : 'after'](this.currentItem);
|
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.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);
|
||||||
}
|
}
|
||||||
}));
|
}));
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user