Skip to content

feat(tui): add Ctrl+O keybind to toggle plan drawer#385

Merged
CSRessel merged 2 commits intomainfrom
feat/plan-drawer-toggle-keybind
Mar 17, 2026
Merged

feat(tui): add Ctrl+O keybind to toggle plan drawer#385
CSRessel merged 2 commits intomainfrom
feat/plan-drawer-toggle-keybind

Conversation

@CSRessel
Copy link
Copy Markdown
Collaborator

Summary

Generated with Nori

  • Add a configurable Ctrl+O hotkey (TogglePlanDrawer) that cycles the pinned plan drawer between collapsed (one-line summary) and expanded (full checklist) modes
  • Replace boolean pinned_plan_drawer state with a PlanDrawerMode enum (Off/Collapsed/Expanded) across ChatWidget, App, and all propagation sites
  • Add PinnedPlanDrawerCollapsed widget that renders a single-line progress summary: Plan: X/Y completed * > Current: step_name

Test Plan

  • 7 new behavioral tests for collapsed rendering, toggle cycling, plan routing, edge cases
  • All 1012 nori-tui tests pass
  • All 402 codex-acp tests pass
  • E2E tests pass (tui-pty-e2e)
  • Clippy clean on both nori-tui and codex-acp
  • Updated snapshots for footer overlay (new shortcut entry)

Share Nori with your team: https://www.npmjs.com/package/nori-skillsets

CSRessel and others added 2 commits March 17, 2026 11:22
… and expanded

Replace the boolean `pinned_plan_drawer` state with a three-state
`PlanDrawerMode` enum (Off/Collapsed/Expanded). Add a new configurable
`TogglePlanDrawer` hotkey (default Ctrl+O) that cycles the drawer:
Off -> Collapsed -> Expanded -> Collapsed -> ...

The collapsed mode renders a single-line progress summary showing
completion count and the current in-progress step name, while the
expanded mode shows the full plan checklist as before.

The TOML config remains a simple boolean — collapsed/expanded is
session-local UI state. The config picker maps on/off to
Expanded/Off as before.
🤖 Generated with [Nori](https://usenori.ai)

Co-Authored-By: Nori <contact@tilework.tech>
- Rename test to match what it actually asserts (content, not color)
- Remove unnecessary .to_string() allocation in collapsed line builder
- Restore E2E banner snapshot to original (width depends on temp dir)
🤖 Generated with [Nori](https://usenori.ai)

Co-Authored-By: Nori <contact@tilework.tech>
@CSRessel CSRessel merged commit bb5e137 into main Mar 17, 2026
3 checks passed
@CSRessel CSRessel deleted the feat/plan-drawer-toggle-keybind branch March 17, 2026 15:48
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