Repository navigation
feat(tui): add an interactive fullscreen transcript scrollbar - #224
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Change
Addresses #216. Enable the fullscreen transcript scrollbar, including track-click jumps and thumb dragging. Reserve a three-column gutter so the larger hit target cannot swallow adjacent content clicks or text selection; narrow terminals retain at least one content column. An overlay cancels an active scrollbar drag on the next mouse event.
Adapted from drowzeys/minimax-code@0b24a7778741fda335a9a2074464c69911feb1aa, submitted under the existing MIT license in #216. Preserve drowzeys as co-author and record the adaptation and Engine file hashes in
LOCAL_CHANGESL035. The original unrestricted three-column hit span failed the added content-click, text-selection and overlay-drag regressions.Validation
pnpm verifyfull profile, all 14 gates on clean commit70bda1ca11f4ed2e7594b1c8073ab7fb659f7839(macOS arm64, Node 26.4.0), including source export, release tools, build, artifact, capabilities, status contract, CLI smoke, offline BYOK, policy and sandbox.Publication and contribution checks
release/public-source.jsonand registered intest/vitest-suites.json.Maintainer handoff
Publication scope or license changes: none.
Shared-source port: pending after public review.
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.