mirror of
https://github.com/jquery/jquery-ui.git
synced 2024-11-21 11:04:24 +00:00
Sortable: Moved change of helper position css. Fixed #5321 – Helper in a bottom-positioned Sortable is positioned wrong on drag start.
This commit is contained in:
parent
a0c734b9e1
commit
32f356bbfa
10
ui/jquery.ui.sortable.js
vendored
10
ui/jquery.ui.sortable.js
vendored
@ -154,11 +154,6 @@ $.widget("ui.sortable", $.ui.mouse, {
|
|||||||
left: this.offset.left - this.margins.left
|
left: this.offset.left - this.margins.left
|
||||||
};
|
};
|
||||||
|
|
||||||
// Only after we got the offset, we can change the helper's position to absolute
|
|
||||||
// TODO: Still need to figure out a way to make relative sorting possible
|
|
||||||
this.helper.css("position", "absolute");
|
|
||||||
this.cssPosition = this.helper.css("position");
|
|
||||||
|
|
||||||
$.extend(this.offset, {
|
$.extend(this.offset, {
|
||||||
click: { //Where the click happened, relative to the element
|
click: { //Where the click happened, relative to the element
|
||||||
left: event.pageX - this.offset.left,
|
left: event.pageX - this.offset.left,
|
||||||
@ -168,6 +163,11 @@ $.widget("ui.sortable", $.ui.mouse, {
|
|||||||
relative: this._getRelativeOffset() //This is a relative to absolute position minus the actual position calculation - only used for relative positioned helper
|
relative: this._getRelativeOffset() //This is a relative to absolute position minus the actual position calculation - only used for relative positioned helper
|
||||||
});
|
});
|
||||||
|
|
||||||
|
// Only after we got the offset, we can change the helper's position to absolute
|
||||||
|
// TODO: Still need to figure out a way to make relative sorting possible
|
||||||
|
this.helper.css("position", "absolute");
|
||||||
|
this.cssPosition = this.helper.css("position");
|
||||||
|
|
||||||
//Generate the original position
|
//Generate the original position
|
||||||
this.originalPosition = this._generatePosition(event);
|
this.originalPosition = this._generatePosition(event);
|
||||||
this.originalPageX = event.pageX;
|
this.originalPageX = event.pageX;
|
||||||
|
Loading…
Reference in New Issue
Block a user