Skip to content

tui: reduce mouse-wheel scroll to ~10% of page height per notch - #656

Merged
glemsom merged 1 commit into
mainfrom
655-mouse-wheel-tenth-page
Aug 29, 2026
Merged

tui: reduce mouse-wheel scroll to ~10% of page height per notch#656
glemsom merged 1 commit into
mainfrom
655-mouse-wheel-tenth-page

Conversation

@glemsom

@glemsom glemsom commented Aug 29, 2026

Copy link
Copy Markdown
Owner

Fixes #655

Summary

Reduces the transcript history mouse-wheel scroll from height/3 per notch to height/10 (with a minimum of 1 row), making wheel navigation gentle and smooth. A physical wheel rotation emits several notch events, so previously a single flick moved ~3 pages.

Changes

  • internal/tui/transcript.go: mouseWheelRows() now scrolls Height()/10 rows per notch (min 1); updated docs on navigateMouse.
  • internal/tui/model_scroll_test.go: renamed TestScroll_mouseWheelMovesByThirdPageTestScroll_mouseWheelMovesTenthOfPage, asserting the new ~10% amount.

Behaviour preserved (verified by existing tests)

  • Wheel up breaks follow and scrolls toward older output; wheel down re-engages follow at the bottom
  • Region routing unchanged: wheel over the history region scrolls, over the band/past-terminal-bottom does not
  • Full internal/tui test suite passes

Note: internal/app and internal/engine test failures in this environment are pre-existing on main (require the bwrap binary which is not installed) and unrelated to this change.

@glemsom
glemsom merged commit 7833ca3 into main Aug 29, 2026
1 check passed
@glemsom
glemsom deleted the 655-mouse-wheel-tenth-page branch August 29, 2026 09:58
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.

Reduce mouse-wheel scroll to ~10% of page height per notch

1 participant