Skip to content

fix(ui): scroll branch tree selections#1058

Merged
abhinav merged 1 commit intomainfrom
fix-branch-tree-selection-scroll
Mar 12, 2026
Merged

fix(ui): scroll branch tree selections#1058
abhinav merged 1 commit intomainfrom
fix-branch-tree-selection-scroll

Conversation

@abhinav
Copy link
Copy Markdown
Owner

@abhinav abhinav commented Mar 12, 2026

Implement scrolling for branch selection prompts
when the terminal cannot show all matching branches.

Add viewport support to fliptree
as the renderer that owns line-by-line tree output,
including scroll marker rendering.
Keep branchtree as the branch-specific adapter
that passes viewport settings through,
and keep BranchTreeSelect responsible for
selection state, filter matching,
and choosing the viewport offset.

Extend the UI script harness
to accept per-script terminal sizes,
and add regression coverage for
constrained-height scrolling
and preserving an already selected branch
when filtering still matches it.

Implement scrolling for branch selection prompts
when the terminal cannot show all matching branches.

Add viewport support to `fliptree`
as the renderer that owns line-by-line tree output,
including scroll marker rendering.
Keep `branchtree` as the branch-specific adapter
that passes viewport settings through,
and keep `BranchTreeSelect` responsible for
selection state, filter matching,
and choosing the viewport offset.

Extend the UI script harness
to accept per-script terminal sizes,
and add regression coverage for
constrained-height scrolling
and preserving an already selected branch
when filtering still matches it.
@abhinav
Copy link
Copy Markdown
Owner Author

abhinav commented Mar 12, 2026

This change is part of the following stack:

Change managed by git-spice.

@abhinav abhinav merged commit 751dd20 into main Mar 12, 2026
22 checks passed
@abhinav abhinav deleted the fix-branch-tree-selection-scroll branch March 12, 2026 20:36
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