The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		May 26, 2026 7:14pm

PR Summary

Low Risk
UI-only preview interaction and SVG display path changes; no auth, API, or data-layer impact.

Overview
Fixes file preview zoom so trackpad pinch and Cmd/Ctrl+scroll zoom the preview content instead of the browser page, by routing wheel handling through a non-passive native listener on the preview viewport.

bindPreviewWheelZoom now treats metaKey like ctrlKey for zoom (macOS Cmd+scroll), and supports an optional onPan callback that consumes two-finger scroll for transform-based viewers instead of relying on container scroll.

ZoomablePreview drops the React onWheel handler and wires zoom/pan through bindPreviewWheelZoom on the viewport ref.

SVG file previews switch from a sandboxed iframe + srcDoc to an <img> fed by a blob: URL (with revoke on cleanup), so pinch/wheel reaches the same zoom layer as other image previews while keeping static SVG display semantics.

^{Reviewed by Cursor Bugbot for commit d0f37e7. Configure here.}

Greptile Summary

This PR fixes trackpad pinch-to-zoom on the file viewer by replacing React's synthetic onWheel (passive by default, silently ignoring preventDefault) with a native non-passive listener via bindPreviewWheelZoom. It also adds metaKey support for Cmd+scroll on macOS and switches SVG previews from a sandboxed iframe to an <img> element backed by a properly lifecycle-managed blob URL.

• preview-wheel-zoom.ts: Adds metaKey alongside ctrlKey for zoom detection and an onPan option that lets transform-based viewers intercept plain scroll events; listener is always registered non-passive so preventDefault() is honored.
• zoomable-preview.tsx: Moves wheel handling from a React onWheel prop to a useEffect-bound native listener — the core fix that makes content zoom instead of browser zoom.
• preview-panel.tsx: Replaces the SVG iframe with a blob-URL <img> using useState+useEffect for correct resource cleanup; security posture is equivalent since SVG loaded as an image cannot execute scripts or load external resources.

Confidence Score: 5/5

Safe to merge — the change is narrowly scoped to wheel-event handling and SVG rendering, with no impact on data, auth, or server-side logic.

All three files make targeted, well-reasoned changes. The move to a non-passive native listener directly fixes the reported bug. The blob-URL lifecycle is handled correctly using useState+useEffect. The only cosmetic issue is the brief broken-image frame from the initial empty src string.

preview-panel.tsx — the initial empty blobUrl state produces a brief broken-image placeholder; minor but worth addressing.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant User as User trackpad
    participant Browser
    participant Viewport as ZoomablePreview
    participant WheelZoom as bindPreviewWheelZoom
    participant ReactState as React state

    User->>Browser: pinch gesture
    Browser->>Viewport: WheelEvent with ctrlKey true
    Note over Viewport,WheelZoom: Non-passive native listener in capture phase
    Viewport->>WheelZoom: onWheel fires
    WheelZoom->>WheelZoom: preventDefault stops browser zoom
    WheelZoom->>ReactState: onZoom callback runs applyZoom
    ReactState->>Viewport: re-render with new zoom transform

    User->>Browser: two-finger pan
    Browser->>Viewport: WheelEvent no modifier key
    Viewport->>WheelZoom: onWheel fires
    WheelZoom->>WheelZoom: onPan present so preventDefault
    WheelZoom->>ReactState: onPan callback runs setOffset
    ReactState->>Viewport: re-render with new offset transform

_{Reviews (2): Last reviewed commit: "fix(files): use effect lifecycle for SVG..." | Re-trigger Greptile}

@greptile

@cursor review