Skip to content

fix(web): Keep canvas zoom out of the detail panel scroll - #68

Merged
4ndreello merged 1 commit into
mainfrom
fix/canvas-chat-scroll-zoom
Sep 8, 2026
Merged

fix(web): Keep canvas zoom out of the detail panel scroll#68
4ndreello merged 1 commit into
mainfrom
fix/canvas-chat-scroll-zoom

Conversation

@4ndreello

Copy link
Copy Markdown
Owner

Scrolling over the open session detail zoomed the canvas instead of scrolling the chat: the window-level wheel handler called preventDefault() on every wheel event, including those bubbling up from #detail and #dChat. The handler now returns early for events inside #detail, so the panel and transcript scroll natively while the canvas keeps zooming everywhere else.

Also makes the chat roomier (400px panel, 52vh transcript with a 140px floor) and styles both scrollbars thin and dark to match the page theme instead of the OS default.

Validated in headless Chromium against a mock session with a long transcript: wheel over chat scrolls with zoom unchanged, wheel over the input leaves zoom alone, wheel over empty canvas still zooms. Scoped suite passes (tests/web-command.test.ts, 28 tests).

The window-level wheel handler zoomed the canvas on every scroll, including over the detail panel, so scrolling the chat zoomed the scene instead. Skip zoom for events inside #detail and let the panel and chat scroll natively.

Also enlarge the chat (400px panel, 52vh transcript) and give the panel scrollbars a thin dark style matching the page theme.
@sonarqubecloud

sonarqubecloud Bot commented Sep 8, 2026

Copy link
Copy Markdown

@4ndreello
4ndreello marked this pull request as ready for review September 8, 2026 18:19
@4ndreello
4ndreello merged commit 8d71844 into main Sep 8, 2026
4 checks passed
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