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

Added: Disable pinch zoom on iOS #1284

Merged
merged 2 commits into from
Sep 24, 2024
Merged

Conversation

drikusroor
Copy link
Contributor

@drikusroor drikusroor commented Sep 24, 2024

This pull request adds a new hook called useDisableIOSPinchZoomOnTouchDevices to the frontend codebase. The hook is responsible for preventing pinch zoom gestures on iOS devices. It is used in the App component to disable pinch zoom on touch devices. Additionally, the pull request includes necessary file changes to implement the hook and add the touch-action property to disable pinch zoom on iOS.

Additionally, the css-property touch-action: manipulation; is added to the body root class in order to prevent touch actions.

Resolves #1274

@drikusroor drikusroor self-assigned this Sep 24, 2024
This commit adds a new hook called useDisableIOSPinchZoomOnTouchDevices to the frontend codebase. The hook is responsible for preventing pinch zoom gestures on iOS devices. It is used in the App component to disable pinch zoom on touch devices.
Copy link
Collaborator

@BeritJanssen BeritJanssen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Unfortunately, cannot test it.

@drikusroor
Copy link
Contributor Author

LGTM. Unfortunately, cannot test it.

Same, I will have to test this on my phone on staging 😅

@drikusroor drikusroor merged commit 6653ef0 into main Sep 24, 2024
7 checks passed
@drikusroor drikusroor deleted the refactor/disable-pinch-zoom branch September 24, 2024 15:36
drikusroor added a commit that referenced this pull request Nov 5, 2024
* fix: correct video links

* fix: Refactor Docker commands in scripts (#1282)

Update the Docker commands in the scripts to use the "docker compose" syntax instead of "docker-compose".

* fix: Fix some search & replace too many, where I mistook `docker-compose-deploy.yml` for a `docker-compose` command (#1283)

* fix(style): Prevent percentage circle from overlapping background circle (#1281)

* Added: Disable pinch zoom on iOS (#1284)

* feat: Add useDisableIOSPinchZoomOnTouchDevices hook

This commit adds a new hook called useDisableIOSPinchZoomOnTouchDevices to the frontend codebase. The hook is responsible for preventing pinch zoom gestures on iOS devices. It is used in the App component to disable pinch zoom on touch devices.

* feat: Add touch-action property to disable pinch zoom on iOS

* fix(style): Move percentage circle down 100% to align with background circle

* fix: use vimeo for videos

* fix: redefine ButtonArray button gaps

* fix: docker compose without hyphen

* fix: update podman-compose file names (#1290)

* adjust debrief template

* Fixed: Fix Circle percentage overlap issue (attempt 3) (#1291)

* style: Move rotate transform and transform origin to the circle's svg level

* story(AutoPlay): Add `AutoPlay` story

* chore(deps): Update `package.json` version to `2.2.5` (#1293)

* fix: remove non-breaking spaces when showing composer / ensemble / piece

* fix: adjust scaling of image in th_preverbal

* chore: update version

* chore: remove unused import

* Refactor: Remove `logoClickConfirm` and make user go to the experiment dashboard directly (#1303)

* refactor: Remove `logoClickConfirm` and make user go to the experiment dashboard directly

* chore(deps): Update package version to 2.2.7

---------

Co-authored-by: BeritJanssen <berit.janssen@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants