Sortable icon indicating copy to clipboard operation
Sortable copied to clipboard

[bug] Pinching to zoom on Android fails occasionally

Open rsmarques opened this issue 4 years ago • 6 comments

When on an Android device, the two-finger pinch action to zoom produces an inconsistent behaviour, often not zooming on the page.

It happens both inside and outside of sortable components.

No need to provide a JSBin, as it's happening on the demo page.

To Reproduce Steps to reproduce the behavior:

  1. Go to https://sortablejs.github.io/Sortable/ on Chrome in an Android device
  2. Perform a two-finger pinch to zoom anywhere on the page.
  3. Page doesn't get zoomed occasionally.

Expected behavior

Page should always get zoomed.

Information

Tested on a Huawei P20 Pro, Android 10, Chrome version 84

rsmarques avatar Aug 19 '20 10:08 rsmarques

It has to do with this touchmove event. I'll see what I can do :S

https://github.com/SortableJS/Sortable/blob/d53f85e7d63ae88d6eb4aa4d7fa3ba110779877c/src/Sortable.js#L2021-L2028

aquaductape avatar Aug 24 '20 04:08 aquaductape

I own a GalaxyA21s using Firefox 79.0.5 and it works fine on my device.

@aquaductape I'm pretty sure that Sortable.active will be null because no sortables are being dragged, so no defaults will be prevented on a zoom in.

waynevanson avatar Aug 31 '20 22:08 waynevanson

@rsmarques Is this on all websites using Sortable, or just the Sortable site?

waynevanson avatar Aug 31 '20 22:08 waynevanson

It occurs on all websites when using Android Chrome. When i remove sortablejs on my project, I'm able to zoom consistently again on Android Chrome.

aquaductape avatar Aug 31 '20 22:08 aquaductape

@waynevanson as @aquaductape stated, it happens in every website on Android Chrome which uses SortableJS

rsmarques avatar Sep 01 '20 03:09 rsmarques

This is still a problem. I recently got rid of JQuery Sortable, and now users are complaing that their gesture doesn't work anymore.

Earlopain avatar Jan 27 '23 18:01 Earlopain