Skip to content

fix(tui): make playbook templates keyboard accessible - #1249

Merged
edwin-zvs merged 2 commits into
mainfrom
fix-playbook-template-keyboard
Aug 13, 2026
Merged

fix(tui): make playbook templates keyboard accessible#1249
edwin-zvs merged 2 commits into
mainfrom
fix-playbook-template-keyboard

Conversation

@edwin-zvs

@edwin-zvs edwin-zvs commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Fixes #1099.

Summary

  • add visible ↑/↓ or C-p/C-n select · Enter apply keyboard affordance to the empty Playbook template list
  • render each template's daemon-provided description beneath its name, matching the web UI
  • keep the full name+description row mouse-activatable
  • update spec 0049 with the keyboard and description semantics

Verification

  • cargo check -p construct-cli
  • cargo test -p construct-cli playbook_empty_template -- --nocapture (2 passed)
  • cargo test -p construct-cli playbook_empty_placeholder -- --nocapture (9 passed)
  • cargo build
  • full cargo test -p construct-cli: 1,395 passed; color::tests::truecolor_backend_still_emits_rgb fails identically on untouched main in this environment

Demo

After-only recording captured from the worktree binary. It opens an empty Playbook, shows descriptions + key hint, focuses Goal with C-n, and applies it with Enter.

@edwin-zvs

edwin-zvs commented Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

After-only TUI demo: descriptions and the visible keyboard hint appear in the empty Playbook; Down focuses Goal and Enter applies it.\n\nissue-1099-playbook-keyboard.mp4

@edwin-zvs

edwin-zvs commented Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

Updated after-only demo: the empty Playbook advertises ↑/↓ or C-p/C-n select · Enter apply; this recording uses C-n to focus Goal and Enter to apply it.\n\nissue-1099-playbook-keyboard.mp4

@edwin-zvs
edwin-zvs merged commit 49b4036 into main Aug 13, 2026
1 check passed
@edwin-zvs
edwin-zvs deleted the fix-playbook-template-keyboard branch August 13, 2026 05:21
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.

tui: playbook empty-state templates are mouse-only and omit the descriptions the web UI shows

1 participant