Skip to content

Comments

Refresh for pinch-to-zoom for macOS trackpad#1555

Merged
ahrm merged 1 commit intoahrm:developmentfrom
degnbol:macos-pinch-zoom
Feb 12, 2026
Merged

Refresh for pinch-to-zoom for macOS trackpad#1555
ahrm merged 1 commit intoahrm:developmentfrom
degnbol:macos-pinch-zoom

Conversation

@degnbol
Copy link

@degnbol degnbol commented Feb 9, 2026

  • Separate pinch gesture from TOUCH_MODE so it works with normal mouse
    behaviour (drag to select, not scroll). TapAndHoldGesture still
    requires TOUCH_MODE.
  • Fix render refresh: set no_rerender only during active pinch, reset
    on gesture end with invalidate_render(). Add validate_render() after
    each zoom step for live visual feedback.
  • Zoom towards cursor position instead of page centre, using the
    existing zoom_in_cursor() which keeps the point under the cursor
    fixed during zoom.

- Separate pinch gesture from TOUCH_MODE so it works with normal mouse
  behaviour (drag to select, not scroll). TapAndHoldGesture still
  requires TOUCH_MODE.
- Fix render refresh: set no_rerender only during active pinch, reset
  on gesture end with invalidate_render(). Add validate_render() after
  each zoom step for live visual feedback.
- Zoom towards cursor position instead of page centre, using the
  existing zoom_in_cursor() which keeps the point under the cursor
  fixed during zoom.
@ahrm
Copy link
Owner

ahrm commented Feb 11, 2026

These should be two different pull requests (and I think the not opening external links makes sense more as a setting and not a command).

@degnbol
Copy link
Author

degnbol commented Feb 11, 2026

These should be two different pull requests (and I think the not opening external links makes sense more as a setting and not a command).

Fixed.

@ahrm ahrm merged commit 7b2b265 into ahrm:development Feb 12, 2026
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