Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Android and delay option #1469

Closed
KingRial opened this issue Mar 13, 2019 · 8 comments
Closed

Android and delay option #1469

KingRial opened this issue Mar 13, 2019 · 8 comments

Comments

@KingRial
Copy link

KingRial commented Mar 13, 2019

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 Chrome Android delay and mousemove events #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

@KingRial KingRial changed the title Android e Delay options Android and delay options Mar 13, 2019
@KingRial KingRial changed the title Android and delay options Android and delay option Mar 13, 2019
@kimemin
Copy link

kimemin commented Apr 3, 2019

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

@owen-m1
Copy link
Member

owen-m1 commented Apr 3, 2019

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

@kimemin
Copy link

kimemin commented Apr 4, 2019

@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
Copy link

clemishow commented Apr 19, 2019

"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
Copy link
Member

owen-m1 commented Apr 19, 2019

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

@clemishow
Copy link

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

@cornucopib
Copy link

@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
Copy link

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

Runi-c pushed a commit to Tupperbox/Sortable that referenced this issue Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants