Skip to content

Keep wheel zoom inside the chart - #4

Merged
callumalpass merged 1 commit into
mainfrom
agent/fix-chart-wheel-zoom
Aug 16, 2026
Merged

callumalpass merged 1 commit into
mainfrom
agent/fix-chart-wheel-zoom

Conversation

@callumalpass

Copy link
Copy Markdown
Owner

What changed

  • replace the React-delegated chart wheel handler with a native non-passive listener
  • cancel Ctrl/Cmd+wheel before Chrome applies page magnification
  • retain pointer-anchored Planner chart zoom
  • leave ordinary wheel scrolling to the browser
  • exercise Ctrl+wheel in the Chromium end-to-end test

Why

Chrome treats React's delegated wheel listener as passive. Planner's existing preventDefault() therefore changed the chart zoom level but could not stop Chrome from magnifying the entire application at the same time.

Impact

Ctrl/Cmd+wheel over the timeline now zooms only the Gantt chart. Normal vertical and horizontal wheel scrolling is unchanged.

Validation

  • pnpm typecheck
  • pnpm lint
  • pnpm test — 37 tests passed
  • pnpm test:e2e — 2 Chromium tests passed
  • pnpm build

@callumalpass
callumalpass merged commit 10d4002 into main Aug 16, 2026
1 check passed
@callumalpass
callumalpass deleted the agent/fix-chart-wheel-zoom branch August 16, 2026 16:06
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