delay option doesn't work

[bhecht2]

Hi,

I am checking the demo: http://jsbin.com/videlojafi/edit?html,js,output

It doesn't work for me on either firefox, or IE (10).
Once removing the delay line, or changing it to 0, everything starts to work

[ranyauto3p]

Hi,

I second that, same experience. Feature is required to allow scrolling on touch devices. With no delay list items gets dragged immediately preventing page scroll.

OK, just found out that if I turn 'forceFallback' on delay works (tested on IE 11 and IPad Safari)

[kpost]

+1. It does work on webkit browsers however, but not on FF.

dragStartFn does get called in the setTimeout, and some stuff is handled, like the classes added to the dragging element. But the actual dragging never happens. Very weird.

[welelay]

doesn't work on Safari on an iPad either

[skitterm]

I found the same issue with IE 11 and Firefox. Setting forceFallback: true solved the issue there (even on iPad Safari for me) but made it so drag-drop doesn't work on Mac Safari (and makes the dragged element look blurry).

[skitterm]

@RubaXa I dug around in the dev code (Sortable.js) and found an issue: When there is a delay for FF/IE11, the dragstart event never fires inside of _triggerDragStart: _on(rootEl, 'dragstart', this._onDragStart);. It looks like the event is attached to the DOM element, but it doesn't fire. I haven't been able to find a fix for it.

[elonmallin]

+1.
If the fixes from a few months ago work maybe they can be merged?

[sbayd]

+1,

It looks this emileber@20a8058 fixes the bug. Is it possible to be merged?

[emileber]

@sbayd I just tried @Xom9lk fix with my own fork and it doesn't really work (weird glitch with different browsers) and It'd need some more work to be totally fixed.

While I'm thankful RubaXa made this project, the way he's handling it now is borderline hostile (closing merge requests and issues without explanations, never replying to some issues, overall tone of the communication, etc.) and it seems like a waste of time to try and get non-trivial pull requests merged. This was opened a year ago and never was acknowledged by him. He clearly doesn't have time for this project unfortunately and we'd better use our own fork.

[RubaXa]

True, I do not have time, but and from you I have not seen any PR.:

https://github.com/RubaXa/Sortable/pulls?utf8=%E2%9C%93&q=is%3Apr%20author%3Aemileber%20

[RubaXa]

@emileber I tried to use your fix, but it does not work (FF): http://recordit.co/PTeUNSLP7g

[RubaXa]

Now the problem can be solved only by means of forceFallback: true.

[emileber]

@RubaXa Thanks for taking time to respond, but like I said in my comment above, my fix doesn't work (in fact, not my fix but Xom9lk's) and after seeing the state of the project, I didn't make any PR.

As of now, I stopped using the delay and might look for a workaround on mobile.