search

SortableJS / vue.draggable.next

Vue 3 compatible drag-and-drop component based on Sortable.js
https://sortablejs.github.io/vue.draggable.next/#/simple
MIT License
3.89k stars 531 forks source link

Dragging selects text on every second drag on Safari #16

Open hphaavikko opened 3 years ago

hphaavikko commented 3 years ago

Dragging selects text on every second drag on Safari (14.0.3). Same issue is reported on older vue Draggable versions as well https://github.com/SortableJS/Vue.Draggable/issues/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 commented 3 years ago

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:

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 commented 3 years ago

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 commented 3 years ago

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 commented 1 year ago

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