Skip to content

feat(ui): wrap output instead of h-scroll - #18

Merged
unrelentingfox merged 1 commit into
mainfrom
feat/wrap-output-viewer
Sep 20, 2026
Merged

unrelentingfox merged 1 commit into
mainfrom
feat/wrap-output-viewer

Conversation

@unrelentingfox

Copy link
Copy Markdown
Owner

Summary

  • Replace horizontal scrolling in the output viewer with ANSI-safe line wrapping (via wrapTextWithAnsi), removing the left/right and h/l hotkeys and their footer hint
  • Move vertical scrolling over visual wrapped rows instead of logical lines, so wrapped continuations of long lines are fully reachable
  • Track wrapped rows with a rowStarts index that rebuilds on resize or log replace and extends incrementally on live appends, keeping follow mode efficient for large logs
  • Update the /bash-async-list README key description

Test plan

  • pnpm run check (216 tests, typecheck, pack check)
  • New tests: wrapping to viewport width; scrolling between wrapped rows of a single long line
  • Reworked the boundary clamp test to vertical-only scrolling
  • Manually verified wrapping with a live async job emitting ~350-char lines

Replace horizontal scrolling in the output viewer with ANSI-safe
wrapping via wrapTextWithAnsi. Vertical scrolling now moves over
visual wrapped rows, tracked by a rowStarts index that rebuilds on
resize or log replace and extends incrementally on live appends.

Remove the left/right/h/l hotkeys and their footer hint; update the
bash-async-list README key description.
@unrelentingfox
unrelentingfox merged commit 61a787b into main Sep 20, 2026
3 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