Skip to content

[codex] Fix keyboard shortcut handling#48

Merged
ctoth merged 2 commits into
masterfrom
codex/keyboard-shortcuts-output-review
Jun 16, 2026
Merged

[codex] Fix keyboard shortcut handling#48
ctoth merged 2 commits into
masterfrom
codex/keyboard-shortcuts-output-review

Conversation

@ctoth

@ctoth ctoth commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Tighten Alt navigation handling so the app only consumes plain Alt navigation chords, not Alt plus extra modifiers.
  • Add output review shortcuts: Ctrl+1 through Ctrl+0 now read the last 1 through 10 output lines.
  • Move sidebar tab switching to Ctrl+Shift+1 through Ctrl+Shift+0.

Root Cause

The global channel-history handler and focused input/output suppressors treated some Alt chords too broadly. Separately, Ctrl+number was already reserved for sidebar tab switching, leaving no direct shortcut for reviewing recent output lines.

Validation

  • npm test -- --run src/hooks/useChannelHistory.test.tsx src/components/input.test.tsx src/components/output.test.tsx src/App.test.tsx
  • npm run typecheck
  • git diff --check on touched files
  • Precommit hooks ran during both commits; typecheck passed. Biome reported existing warnings in App.tsx and output.tsx, but did not block.

@ctoth ctoth merged commit e88d491 into master Jun 16, 2026
1 check passed
github-actions Bot pushed a commit that referenced this pull request Jun 16, 2026
…ut-review

[codex] Fix keyboard shortcut handling e88d491
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