Skip to content

I've noticed that using pinch to zoom on mobile seems to ...#1785

Merged
lukemarsden merged 1 commit intomainfrom
feature/001411-ive-noticed-that-using
Mar 2, 2026
Merged

I've noticed that using pinch to zoom on mobile seems to ...#1785
lukemarsden merged 1 commit intomainfrom
feature/001411-ive-noticed-that-using

Conversation

@lukemarsden
Copy link
Collaborator

Helix: I've noticed that using pinch to zoom on mobile seems to be granier than it reasonably should be given the resolution

Issue 1: Pinch-to-zoom quality
- Changed zoom implementation from CSS transform scale() to CSS width/height scaling
- This allows the browser to render more pixels from the native resolution canvas buffer
- Added overflow:hidden to container to clip zoomed content

Issue 2: Trackpad edge-pan not working
- Fixed coordinate system mismatch in edge detection
- Now uses container edges directly since cursor is in container-relative coords
- Removed unused calculateVisibleViewportBounds() function

Spec-Ref: helix-specs@e2e2bcb22:001411_ive-noticed-that-using
@lukemarsden lukemarsden merged commit 839a4c0 into main Mar 2, 2026
3 checks passed
@lukemarsden lukemarsden deleted the feature/001411-ive-noticed-that-using branch March 2, 2026 08:33
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.

1 participant