Android and delay option

SortableJS / Sortable

Reorderable drag-and-drop lists for modern browsers and touch devices. No jQuery or framework required.

https://sortablejs.github.io/Sortable/

MIT License

29.63k stars 3.7k forks source link

Android and delay option #1469

Closed KingRial closed 5 years ago

KingRial commented 5 years ago

Problem:

I am testing the library with both iOS and Android. On iOS seems perfect; however on Android, when using the library with the "delay" option, it's very difficult to use.

There are two specific situations:

Try just to touch to start the sortable; with delay set I neraly always fail to start the sortable; without the delay options is perfect (could it be related to https://github.com/SortableJS/Sortable/issues/981 ?)
If you use an high delay (for example 500) after some simple repeated scrolls, you need to wait 2x delay to see the sortable work (it's like if when scrolling the delay launched from the touchstart triggered by the scroll is not cancelled)

Chrome: 72.0.3626.121 Android 8.1.0

JSBin/JSFiddle demonstrating the problem:

https://jsbin.com/nolazet

kimemin commented 5 years ago

Delay option in Android is not working again!! I want to need help, too.

owen-m1 commented 5 years ago

@kimemin Was there a version in which it did work?

kimemin commented 5 years ago

@owen-m1 I found well-working version, but old one. I used https://github.com/SortableJS/Sortable/tree/584f5e52d0ca13fc03b9a4325521352c304c4647. this version works on both Android and iOS.

clemishow commented 5 years ago

"Try just to touch to start the sortable; with delay set I neraly always fail to start the sortable; without the delay options is perfect (could it be related to #981 ?)" Exactly got the same problem in 1.8.4 with this options :

const sortableOptions = {
    forceFallback: true,
    fallbackOnBody: true,
    animation: 150,
    delay: 100,
    filter: '.static',
    ghostClass: 'ghost',
    fallbackClass: 'chosen',
    chosenClass: 'chosen',
    scrollSpeed: 20,
  };

But now IOS Safari delay not working with #584f5e52d0ca13fc03b9a4325521352c304c4647

owen-m1 commented 5 years ago

@KingRial @kimemin @clemishow Please try master branch, and let me know if it solved the problem on your end.

clemishow commented 5 years ago

Working for me, thank you very much for your reactivity 👏

cornucopib commented 5 years ago

@owen-m1 I found well-working version, but old one. I used https://github.com/SortableJS/Sortable/tree/584f5e52d0ca13fc03b9a4325521352c304c4647. this version works on both Android and iOS.

very good!!!

AndyMachinCG commented 1 year ago

I was able to fix this issue by setting the draggable option. draggable: ".item"