Skip to content

fix(tui): hide the overlay select marker and scroll indicator - #1456

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-10639-tui-adopt-select-tabselect-delete-bespoke-overlay-list
Oct 16, 2026
Merged

TheGreatAxios merged 1 commit into
mainfrom
cl-10639-tui-adopt-select-tabselect-delete-bespoke-overlay-list

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Oct 16, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Overlay choices stay on the native Select. The selection marker and the scroll indicator stay off, including after a height rebuild.
  • Selection is still color only. Empty lists still read (no choices).

Verification

  • bun run check passes (9123 tests, 0 fail)
  • bun test ./src/tui/overlay-list.test.ts ./src/tui/overlay-empty-state.test.ts ./src/tui/overlay-overflow.test.ts ./src/tui/description-zone.test.ts ./src/tui/decision-truncation.test.ts ./src/tui/allow-once-reprompt.test.ts --randomize --seed 424242 passes (61 tests)

Part of CL-10639

@linear-code

linear-code Bot commented Oct 16, 2026

Copy link
Copy Markdown

CL-10639

@corbits-triage

corbits-triage Bot commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

@TheGreatAxios TheGreatAxios changed the title feat(tui): paint overlay choices with native Select fix(tui): hide the overlay select marker and scroll indicator Oct 16, 2026
@TheGreatAxios
TheGreatAxios merged commit b013689 into main Oct 16, 2026
13 checks passed
@TheGreatAxios
TheGreatAxios deleted the cl-10639-tui-adopt-select-tabselect-delete-bespoke-overlay-list branch October 16, 2026 05:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant