Skip to content

Decouple sidebar from layout toggle#18

Merged
benvinegar merged 1 commit intomasterfrom
decouple-sidebar-from-layout
Mar 21, 2026
Merged

Decouple sidebar from layout toggle#18
benvinegar merged 1 commit intomasterfrom
decouple-sidebar-from-layout

Conversation

@benvinegar
Copy link
Member

Summary

  • Layout toggles (1/2/0) now only change split/stack diff rendering — they no longer collapse the sidebar
  • New s keybinding and View > Sidebar menu item to independently toggle the sidebar
  • Sidebar visibility is based on terminal width (≥220 cols) and the new toggle, not layout mode

Test plan

  • bun run typecheck passes
  • bun test passes (1 pre-existing unrelated failure)
  • Smoke test: press 1/2 on a wide terminal and verify sidebar stays
  • Smoke test: press s to toggle sidebar on/off

🤖 Generated with Claude Code

Layout toggles (1/2/0) now only change the diff view between split/stack
without collapsing the sidebar. Sidebar visibility is controlled by a new
independent toggle: press `s` or use View > Sidebar in the menu.

On wide terminals (≥220 cols), the sidebar now stays visible regardless of
which layout mode is active. On narrower terminals it remains hidden as
before since there isn't enough room.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@benvinegar benvinegar force-pushed the decouple-sidebar-from-layout branch from 4b50dd9 to 75bd3c4 Compare March 21, 2026 15:45
@benvinegar benvinegar merged commit cd14a61 into master Mar 21, 2026
9 checks passed
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