Skip to content

Smooth fullscreen transcript wheel scrolling - #887

Open
mrmoxon wants to merge 4 commits into
mainfrom
improvements/smoother-fullscreen-scroll
Open

Smooth fullscreen transcript wheel scrolling#887
mrmoxon wants to merge 4 commits into
mainfrom
improvements/smoother-fullscreen-scroll

Conversation

@mrmoxon

@mrmoxon mrmoxon commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • reduce fullscreen transcript wheel scrolling from three rows per report to one row per report
  • preserve immediate movement on every wheel input
  • keep the viewport aligned to complete terminal rows
  • update the TUI changelog and regression coverage

Why

Fullscreen transcript scrolling was too sensitive: one wheel report jumped three rows. One row per report is the terminal renderer’s finest visible increment and avoids both the original large jumps and the over-throttled two-report behavior.

Testing

  • TUI test suite: 737 passed
  • repository pre-commit checks, including formatting, TypeScript, and browser/installer smoke checks

Note

Low Risk
Single constant change in fullscreen mouse wheel handling with matching test updates; no auth, data, or API surface changes.

Overview
Fullscreen transcript wheel scrolling is less sensitive: each mouse wheel report advances the viewport by one terminal row instead of three (WHEEL_SCROLL_LINES in tui.ts).

Wheel handling still runs on every report (no extra throttling), and row alignment is unchanged. The TUI changelog documents the behavior, and fullscreen regression tests were updated for the new scroll distance and linesBelow counts.

Reviewed by Cursor Bugbot for commit b04ed8b. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Reduce fullscreen transcript wheel scroll sensitivity from 3 lines to 1 line per step

Decreases TUI.WHEEL_SCROLL_LINES from 3 to 1 in tui.ts so the viewport moves one row per wheel step instead of three, making fullscreen transcript scrolling feel smoother. Test assertions in fullscreen.test.ts are updated to match the new scroll increment and expected linesBelow count.

Macroscope summarized b04ed8b.

Comment thread packages/tui/src/tui.ts
@mrmoxon
mrmoxon marked this pull request as ready for review August 8, 2026 00:33
zhengr pushed a commit to zhengr/prime-agent that referenced this pull request Aug 8, 2026
…t-ai#887)

Error messages for missing credentials and shell config now use
getAuthPath() and getSettingsPath() instead of hardcoded ~/.pi/agent/
paths, so they correctly reflect PI_CODING_AGENT_DIR when set.
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