BACK-633 - Scroll the task detail pane with Shift+J/K without leaving the list - #771
Open
kossakovsky wants to merge 1 commit into
Open
BACK-633 - Scroll the task detail pane with Shift+J/K without leaving the list#771kossakovsky wants to merge 1 commit into
kossakovsky wants to merge 1 commit into
Conversation
kossakovsky
force-pushed
the
tasks/back-537-detail-pane-scroll-keys
branch
from
July 22, 2026 14:10
05b6b63 to
d817464
Compare
… the list In the task-list view the detail pane can only be scrolled after moving focus into it (right/l or Enter). Bind Shift+J / Shift+K as screen-level shortcuts that scroll the detail pane body from either pane, so the list keeps focus while long descriptions are read. The shortcut is inert while filters are focused, a popup/modal is open, or there is no detail pane. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
kossakovsky
force-pushed
the
tasks/back-537-detail-pane-scroll-keys
branch
from
August 11, 2026 17:09
d817464 to
5707c2b
Compare
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.
Summary
In the task-list view the detail pane can only be scrolled after moving focus into it (
right/lorEnter). This bindsShift+J/Shift+Kas screen-level shortcuts that scroll the detail pane body from either pane, so the list keeps focus while long descriptions are read.j/klist navigation is untouched (uppercase-only bindings, following the existing['e','E','S-e']idiom)Rebased onto current main
Rebased from the July branch point onto current
main. Two notes on how this interacts with what shipped since:kat the top of the detail pane already stays put. The remaining scope of this PR is only the scroll-without-leaving-the-list behaviour. The new helper composes with the currentshouldMoveFromDetailBoundaryToSearch(scrollOffset, key)signature and adds no interaction with it; the existing navigation suites pass unchanged.mainallocated that ID to "Make checklist edits and serialization deterministic" on 2026-07-11. To avoid introducing a duplicate task ID, the task was recreated through the CLI allocator as BACK-633 and the old file dropped. The branch name still saysback-537because renaming it would close this PR.Related Issue or Task
Closes #769
Backlog task: BACK-633
Task Checklist
backlog/tasks/Testing
bunx tsc --noEmit— cleanbun run check .— clean (374 files)bun run test— 2250 passed / 6 skipped / 1 failed. The single failure issrc/test/tui-window-title.test.ts, which fails identically on unmodifiedmainin my environment (I run tests inside tmux, which rewrites the terminal escape sequences that test asserts on), so it is pre-existing and unrelated to this change.src/test/task-viewer-detail-scroll.test.tscovers all six branches of the new helper.viewTaskEnhancedis not exercised by the test suite: fiveShift+Jscrolled the detail body without changing the selected task, threeShift+Kscrolled back, and a subsequent plainjmoved the selection — confirming focus never left the list. With the search field focused,Jwas inserted as a literal character and the detail pane did not scroll.Happy to add a key-event test in the style of
src/test/tui-vim-boundary-navigation.test.tsif you'd prefer that behaviour locked down in CI rather than verified by hand.