1.13.0 bug in sortable, draggable: freezes page, "this.dragDirection" is undefined about jquery-ui HOT 11 CLOSED

Thanks for the report. Can you provide a test case on JS Bin? Thanks!

from jquery-ui.

@danbernier ping

from jquery-ui.

I'll take a crack at it later today.

from jquery-ui.

@danbernier A gentle reminder about a test case. :)

from jquery-ui.

@danbernier Another friendly ping. :)

from jquery-ui.

Here is a demo showing the issue using nestedSortable and their default example:
https://jsbin.com/mizadimobu/2/edit?html,css,js,output

With jquery ui 1.12.1 it works fine, on 1.13.0 it throws the error. Sorry I don't have a simpler example for you, this is just one that we ran into, forcing us to rollback to 1.12.1 for now.

from jquery-ui.

Thanks @jwcooper! Sorry this got away from me @mgol. @jwcooper's looks about like what mine would've been.

from jquery-ui.

@jwcooper From what I see, the jQuery UI Nested Sortable library overwrites some methods in $.ui.sortable.prototype, in particular the private _mouseDrag method. When you do that, all bets are off. And, indeed, their version of _mouseDrag doesn't set the dragDirection property which is then used in other methods. That needs to be fixed in jQuery UI Nested Sortable.

@jwcooper are you by any chance using the same jQuery UI Nested Sortable library? Ah, I see from the stack trace that you do.

from jquery-ui.

I don't think there's anything to be done on the jQuery UI side here. The bug is in the jQuery UI Nested Sortable library, the way it overwrote private parts of the Sortable widget has always been risky. The library doesn't look maintained, with the last commit being done 4 years ago. Your best bet may be forking the library and backporting jQuery UI 1.13.0 changes to it. For this specific issue, the most important is this bit of _mouseDrag: https://github.com/jquery/jquery-ui/blob/1.13.0/ui/widgets/sortable.js#L442-L445. I can't guarantee there won't be more changes to backport until it stops being broken, though.

I'm going to close this since the issue is not in UI but feel free to continue discussing the issue here.

from jquery-ui.

@mgol Thanks for the tip on the issue and the fix, really appreciate it. It was indeed nested sortable that broke, and adding the dragDirection (and a few more things) worked with 1.13.

from jquery-ui.

This is fixed in ilikenwf/nestedSortable#138 (comment)
But the problem is it is not released yet(v2.1a). You can manually take the source code.

from jquery-ui.