Skip to content

Conversation

@Acters
Copy link

@Acters Acters commented Feb 12, 2026

This PR fixes an off-by-one rendering issue in the resume modal where one selectable session row could be hidden.

When a non-auto modal view (like the resume picker) was rendered, the bottom pane still reserved composer padding, which reduced the visible view height by one row. That caused:

  • A single resumable session to be selectable but not visibly rendered.
  • N resumable sessions to show only N-1 visible rows.

What changed

  • Removed bottom-padding subtraction for non-auto modal views so they consume the full available modal height.
  • Kept existing behavior for auto coordinator views and composer layout.
  • Added a regression test that verifies a single resume row is visible at desired height.

Files changed

  • code-rs/tui/src/bottom_pane/mod.rs

Validation

  • cargo test -p code-tui bottom_pane::tests::resume_selection_renders_single_row_at_desired_height

fixes #542

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.

/resume picker shows one fewer row than available sessions (1 session => 0 visible rows)

1 participant