Sorting Breaks with jQuery UI 1.13+

[FritzTGreen]

This closed issue in the jQuery UI repository confirms and describes the issue.

Essentially, if using jQuery UI 1.13 or higher, attempting to drag an item in a nested sortable list freezes the list.

This issue does not happen with jQuery UI 1.12.

Is anybody maintaining a fork in which this issue has been resolved?

Thanks!
Fritz

[rubenschulz]

Hi Fritz,

I had the same issue as you, this seems to be broken in jquery ui, not in this tool.
I managed to fix this by changing 2 lines in the jquery-ui.js file:

On line 16164 and 16165, replace those line:
verticalDirection = this.dragDirection.vertical;
horizontalDirection = this.dragDirection.horizontal;

with:
verticalDirection = this._getDragVerticalDirection();
horizontalDirection = this._getDragHorizontalDirection();

Hope this helps!

[FritzTGreen]

Thanks Ruben! I appreciate your help very much! Fritz Green 918-213-5497 ***@***.*** Bartlesville, OK

…

On Feb 9, 2022, at 2:39 PM, Ruben Schulz ***@***.***> wrote: Hi Fritz, I had the same issue as you, this seems to be broken in jquery ui, not in this tool. I managed to fix this by changing 2 lines in the jquery-ui.js file: On line 16164 and 16165, replace those line: verticalDirection = this.dragDirection.vertical; horizontalDirection = this.dragDirection.horizontal; with: verticalDirection = this._getDragVerticalDirection(); horizontalDirection = this._getDragHorizontalDirection(); Hope this helps! — Reply to this email directly, view it on GitHub <#138 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AB2OR6EGG3UFKBVXO6XMBSTU2LGHRANCNFSM5M6WXTBA>. Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you authored the thread.

[codertushar]

This is fixed in 2.1a version of current library:
https://github.com/ilikenwf/nestedSortable/blob/master/jquery.mjs.nestedSortable.js#L266-L269

Changing in jquery-ui as suggested in above comments is not required.

@FritzTGreen Please release it in a new tag as also requested in #141

[ucjonathan]

@rubenschulz Thank you for posting that fix. Dealing with that exact issue right now!