react-zoom-pan-pinch icon indicating copy to clipboard operation
react-zoom-pan-pinch copied to clipboard

Zoom doesn't work properly on mobile when both fingers touch the screen simultaneously

Open MarwanEsm opened this issue 1 year ago • 18 comments

Describe the bug Zoomin-In/Out won't work properly when a user touches a mobile device's screen with two fingers at the same time,

Expected behavior Immedita response and smoth Zooming-In/Out

Smartphone (please complete the following information):

  • Device: iPhone 13, tablet
  • Browser : safar, chrom

MarwanEsm avatar Jul 03 '24 13:07 MarwanEsm

We are having the same Issue since this commit: https://github.com/BetterTyped/react-zoom-pan-pinch/commit/9525dbbae52f79aec452ad71e597cc830f15426b

We use this Dependency inside of Electron and embedded Chrome.

    "electron": "30.0.9",

nikbrunner avatar Jul 03 '24 14:07 nikbrunner

Can confirm this is an issue here too since i just updated.

sanderkooger avatar Jul 03 '24 17:07 sanderkooger

Thanks, will it be resolved soon ?

MarwanEsm avatar Jul 04 '24 05:07 MarwanEsm

I had the same problem with a 3.6.1 version, I changed to 3.4.4 and it worked well. Something changed between that two version.

bethuxs avatar Jul 06 '24 03:07 bethuxs

I'm also having pinch problem on 3.6.1 with Android Chrome 126.0.6478.122. 3.6.0 was fine though.

yhuang1211 avatar Jul 08 '24 08:07 yhuang1211

We're back on 3.4.4 as well to avoid this issue for our project.

hawkeye217 avatar Jul 17 '24 12:07 hawkeye217

Going back to 3.4.4 solved this problem, thanks!!

init1-dev avatar Jul 18 '24 01:07 init1-dev

Pinch zoom didn't work at all in android when I was using ^3.6.1 and going back to older versions fixed it.

krishnakiriti04 avatar Aug 01 '24 07:08 krishnakiriti04

Important note: when rolling back from 3.6.1 to 3.6.0, entry in package.json should be: "react-zoom-pan-pinch": "3.6.0", NOT "react-zoom-pan-pinch": "^3.6.0", I have a generalized idea of why this is the case, but would appreciate if someone could offer a specific explanation.

Another note: creating a test app using npx create-react-app my-app will also produce failure of pinch-zoom on mobile.

txtjqm avatar Aug 01 '24 11:08 txtjqm

I encountered the same issue, so I submitted a PR to fix the issue. https://github.com/BetterTyped/react-zoom-pan-pinch/pull/499

suke avatar Aug 10 '24 08:08 suke

I'm also having pinch problem

iamhuynq avatar Aug 23 '24 07:08 iamhuynq

We are using the fork now for a while, seems to work fine.

bhauteki avatar Aug 23 '24 07:08 bhauteki

Rolled back to 3.6.0 and it works just fine. Keep in mind that you have to write it without "^" and just in case reinstall node modules

mustafagarko avatar Oct 08 '24 14:10 mustafagarko

still having the same issue even with version 3.6.0 or 3.4.4 😞 is there a plan to fix this?

leonardomorenozaga avatar Nov 27 '24 19:11 leonardomorenozaga

Important note: when rolling back from 3.6.1 to 3.6.0, entry in package.json should be: "react-zoom-pan-pinch": "3.6.0", NOT "react-zoom-pan-pinch": "^3.6.0", I have a generalized idea of why this is the case, but would appreciate if someone could offer a specific explanation.

Another note: creating a test app using npx create-react-app my-app will also produce failure of pinch-zoom on mobile.

I am having an old CRA app and this is true the pinch doesn't work immediately on mobile! Rolling back to 3.6.0 fixed the issue for me..

pranavgoel29 avatar Jan 24 '25 08:01 pranavgoel29

I encountered the same issue, so I submitted a PR to fix the issue. #499

This worked fine for me!

Why this is not integrated yet? This bug is 7 months old! @prc5, please, It's a very simple fix! 🙏

davideramoaxa avatar Jan 31 '25 09:01 davideramoaxa

Can we confirm this bug is fixed by #499 ?

davideramoaxa avatar Feb 06 '25 15:02 davideramoaxa

I can confirm version 3.7.0 fixes my zoom issue!

davideramoaxa avatar Feb 12 '25 14:02 davideramoaxa