jquery-sortable icon indicating copy to clipboard operation
jquery-sortable copied to clipboard

Fix touch support.

Open taliastocks opened this issue 9 years ago • 6 comments

Fix touch support. In case we are dragging something clickable, preventDefault on drag, not on mousedown/touchstart.

taliastocks avatar Mar 12 '15 13:03 taliastocks

I have not recompiled the minified version of the script. I assume you have a process for doing that.

taliastocks avatar Mar 12 '15 13:03 taliastocks

Hi, thanks for the patch but I've got a problem with it:

On touch devices like iPad or Android I experienced a problem: I've got the following markup: ul li a href=xyz

Clicking on links works now with your patch, also the dragging works, but when dropping the link is clicked instead of firing the onDrop event.

Any idea?

klemens-u avatar May 07 '15 06:05 klemens-u

Hmm, I'm not sure. Feel free to investigate. I didn't end up using this library for the feature I was working on, after all, so I don't have a lot invested in debugging this anymore.

taliastocks avatar May 11 '15 15:05 taliastocks

Hi. This is great Fix! I was successful it on FirefoxOS.

ycatch avatar Jan 09 '16 07:01 ycatch

If anyone is having this error in console : [Intervention] Unable to preventDefault inside passive event listener due to target being treated as passive

Just add touch-action: none; to the css of the ul or ol to fix it

devmotionfr avatar Mar 04 '21 18:03 devmotionfr

@devmotionfr I've used html { touch-action: manipulation; } to do it for the whole page. I don't remember exactly why though... And just added a comment that touch-action: pan-x will prevent the whole page to be scrolled (that should have been a solution for this warning when it first appeared).

dugwood avatar Mar 04 '21 18:03 dugwood