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

Dragging selects text on every second drag on Safari #16

Open
hphaavikko opened this issue Feb 18, 2021 · 4 comments
Open

Dragging selects text on every second drag on Safari #16

hphaavikko opened this issue Feb 18, 2021 · 4 comments

Comments

@hphaavikko
Copy link

hphaavikko commented Feb 18, 2021

Dragging selects text on every second drag on Safari (14.0.3). Same issue is reported on older vue Draggable versions as well SortableJS/Vue.Draggable#743 but the mentioned forceFallback option didn't work.

You can reproduce the issue by trying the live demo with Safari: https://sortablejs.github.io/vue.draggable.next/#/simple Drag once and it works, drag a second time and it selects text. Drag third time and it works, fourth time it selects text and so on.

It tried setting user-select: none to the item css (with all prefixes as well) but it doesn't help; then Safari just starts selecting text from the first selectable element before the draggable list.

@ctessmer
Copy link

This problem happens on iOS Safari too.

Every second tap and drag selects text—just like @hphaavikko described—instead of dragging the item. Tested (and failed) on:

  • macOS 10.15.7; Safari 14.0.1
  • iOS 14.4; Safari 14

I tested it on Firefox Developer Edition 87.0b5. It works like a charm.

Awesome library. Really hope Apple users get some love here. I mean, iPhones 📱 seem to be somewhat popular… 😉

@ahoiroman
Copy link

Seems like forceFallback didn't make it to vuedraggable@next. Would love to see this as I can't migrate to Vue3 with this behavior on Safari.

@apodacaduron
Copy link
Contributor

apodacaduron commented Jun 29, 2021

This was fixed in SortableJS https://github.com/SortableJS/Sortable/releases/tag/1.13.0, but I have no idea why they haven't updated the package yet, this library still in v1.10 of SortableJS 😭, sent this PR to see if it gets approved PR

@HanKruiger
Copy link

It looks like this can be closed now, as vue.draggable.next uses SortableJS 1.14 and I can no longer reproduce the issue.

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

5 participants