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

[PLAY-1601] Bump react-zoom-pan-pinch from 2.6.1 to 3.6.0 #3817

Merged
merged 3 commits into from
Oct 22, 2024

Conversation

kangaree
Copy link
Contributor

@kangaree kangaree commented Oct 17, 2024

What does this PR do?
Upgrade react-zoom-pan-pinch from 2.6.1 to 3.6.0

Fix "pinch" bug on mobile and tablet by avoiding 3.6.1 version which has a bug with "double click" and "two fingers" on zoom.
See discussion in GitHub issues.
Someone made a PR which hasn't been merged yet, so this might be fixed in later versions.

How to test?
Go through the Lightbox kit for any regressions

Checklist:

  • LABELS Add a label: enhancement, bug, improvement, new kit, deprecated, or breaking. See Changelog & Labels for details.
  • DEPLOY I have added the milano label to show I'm ready for a review.

@kangaree kangaree added improvement This is used when your PR contains library upgrades or doc/site improvements. (USED IN CHANGELOG)) alpha labels Oct 17, 2024
@kangaree kangaree requested review from a team as code owners October 17, 2024 17:24
Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 14.5.0.pre.alpha.PLAY1601updatereactzoompanpinch4123

Your Alpha for NPM is 14.5.0-alpha.PLAY1601updatereactzoompanpinch4123

BetterTyped/react-zoom-pan-pinch#487
Double click causes issues on mobile for two fingers
@kangaree kangaree changed the title [PLAY-1601] Bump react-zoom-pan-pinch from 2.6.1 to 3.6.1 [PLAY-1601] Bump react-zoom-pan-pinch from 2.6.1 to 3.6.0 Oct 17, 2024
@kangaree kangaree added alpha and removed alpha labels Oct 17, 2024
Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 14.5.0.pre.alpha.PLAY1601updatereactzoompanpinch4128

Your Alpha for NPM is 14.5.0-alpha.PLAY1601updatereactzoompanpinch4128

@kangaree kangaree added the milano 20 MAX - Deploy this PR to a review environment via Milano label Oct 18, 2024
@nidaqg nidaqg added Product Approved pending technical review, OK to merge to master Code Approved Approved by a Playbook Admin labels Oct 22, 2024
@nidaqg nidaqg added this pull request to the merge queue Oct 22, 2024
Merged via the queue into master with commit abf83a5 Oct 22, 2024
8 checks passed
@nidaqg nidaqg deleted the PLAY-1601-update-react-zoom-pan-pinch branch October 22, 2024 19:43
Copy link

You merged this pr to master branch:
- Ruby Gem: 14.6.0.pre.rc.24
- NPM: 14.6.0-rc.24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alpha Code Approved Approved by a Playbook Admin improvement This is used when your PR contains library upgrades or doc/site improvements. (USED IN CHANGELOG)) milano 20 MAX - Deploy this PR to a review environment via Milano Product Approved pending technical review, OK to merge to master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants