bds-cli tui has weird behaviour when using j/k to move the focus of a list in the sidebar - the sidebar does not scroll #128

Closed
opened 2026-08-05 20:36:27 +00:00 by hugo · 1 comment
Owner

when using j/k to move on a list of things in any sidebar, the sidebar should scroll to accomodate the position of the current element in the sidebar that is selected. right now the selection mark just moves out of the screen space, which is not helping the user. the sidebar needs to scroll to follow the selection.

when using j/k to move on a list of things in any sidebar, the sidebar should scroll to accomodate the position of the current element in the sidebar that is selected. right now the selection mark just moves out of the screen space, which is not helping the user. the sidebar needs to scroll to follow the selection.
hugo added the bug label 2026-08-05 20:36:27 +00:00
Author
Owner

Implemented in 0417aaf.\n\nThe shared TUI sidebar renderer now uses Ratatui's stateful List/ListState, so every sidebar viewport follows selected_index for j/k and arrow navigation while retaining existing header and focus styling. Added an Allium visibility guarantee, a j/k regression covering downward and upward scrolling in a constrained viewport, and updated the README.\n\nReviewed independently against the issue, bDS2's stateful sidebar list, and specs/tui.allium. Verified with focused red/green testing, Allium check/analyse (zero findings), fmt, warnings-denied workspace Clippy, the full workspace test suite (all passing; one existing ignored real-model test), workspace build, release macOS bundle, and a real bundled TUI session over a long list in both directions.

Implemented in 0417aaf.\n\nThe shared TUI sidebar renderer now uses Ratatui's stateful List/ListState, so every sidebar viewport follows selected_index for j/k and arrow navigation while retaining existing header and focus styling. Added an Allium visibility guarantee, a j/k regression covering downward and upward scrolling in a constrained viewport, and updated the README.\n\nReviewed independently against the issue, bDS2's stateful sidebar list, and specs/tui.allium. Verified with focused red/green testing, Allium check/analyse (zero findings), fmt, warnings-denied workspace Clippy, the full workspace test suite (all passing; one existing ignored real-model test), workspace build, release macOS bundle, and a real bundled TUI session over a long list in both directions.
hugo closed this issue 2026-08-07 13:32:30 +00:00
Sign in to join this conversation.