Sortable: jump when moving element

This appears to be an issue in Sortable's _mouseStart().

Here, the sortable helper is absolutely-positioned before the placeholder is created/inserted. This results in a "jump" (as viewed from page bottom) because the document height shrinks when the helper is abs-positioned. One solution: create the placeholder before the helper is abs-positioned. I did a quick test; seems to fix the "jump".

I second that,

issue with scroll jump can be removed simply by moving 'this._createPlaceholder();' from line 185(jquery.ui.sortable.js) just before line 156, where helper gets position:absolute.

I have no clue if it affects further offset calculations, but it seems to work for my case with nested sortables.

Other way is to set height of currentItem's parent just before repositioning helper and unset that after creation of placeholder.

Thanks for taking the time to contribute to the jQuery UI project! I can no longer reproduce the issue using the latest jQuery and jQuery UI using the info provided. http://jqueryui.com/sortable

If you can still reproduce it, please feel free to reply to this ticket with a test case showing the problem. Thanks!

Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!

I just tested this with current jquery ui 1.10.3 and for me the issue still exists.

It is easily fixed by the suggestions above (by moving this._createPlaceholder(); before the absolute positioning of the helper).

As far as i can see, the change doesn't brake anything, so please patch this.

Should i just add a pull request to the github repo?

Replying to [comment:7 habermann24]:

I just tested this with current jquery ui 1.10.3 and for me the issue still exists. It is easily fixed by the suggestions above (by moving this._createPlaceholder(); before the absolute positioning of the helper). As far as i can see, the change doesn't brake anything, so please patch this. Should i just add a pull request to the github repo?

Hi habermann24,

We need a reduced test case showing the issue in order to look into this. You can use this to get started http://jsfiddle.net/tj_vantoll/YMnwU/.

I could reproduce this on a project I've been working on.

You can check it out on http://jsfiddle.net/BDNVK/

If you scroll on the list and the try to reorder, the element jumps top. It is achieved by the combination of overflow:auto on the body and position:relative on the container.

Hope it helps...

Replying to [comment:9 nsimoes]:

I could reproduce this on a project I've been working on. You can check it out on http://jsfiddle.net/BDNVK/ If you scroll on the list and the try to reorder, the element jumps top. It is achieved by the combination of overflow:auto on the body and position:relative on the container. Hope it helps...

Thanks nsimoes. That problem is covered by #9588, but your test case helps.

The problem still exist...