Skip to content

feat(chat): Ctrl+Enter to start session, arrow-key model nav (T-51) - #17

Merged
jaesbit merged 1 commit into
mainfrom
feat/t51-ctrl-enter-start
Jul 4, 2026
Merged

feat(chat): Ctrl+Enter to start session, arrow-key model nav (T-51)#17
jaesbit merged 1 commit into
mainfrom
feat/t51-ctrl-enter-start

Conversation

@jaesbit

@jaesbit jaesbit commented Jul 3, 2026

Copy link
Copy Markdown
Owner

Why

T-51: the "New session" modal (SessionLaunchModal) had no keyboard shortcut to submit and no way to browse the model list without the mouse.

Scope

  • Ctrl+Enter (also Cmd+Enter) anywhere in the modal starts the session, mirroring the primary button.
  • Arrow-key navigation (↑↓) through the model list in the search input, with Enter to pick the highlighted entry. Mouse hover stays in sync with the keyboard highlight.

Surface area

apps/web/src/components/chat/SessionLaunchModal.tsx only. ModelPickerModal.tsx (switch-model-on-existing-session) is untouched — out of scope per the ticket.

Verification

  • bun run --filter '@omp-deck/web' typecheck — pass
  • bun run --filter '@omp-deck/web' build — pass
  • No React Testing Library precedent exists anywhere in this repo (confirmed via search — zero .test.tsx files, no @testing-library/* dependency), so no component test was added; traced the logic manually instead.

@jaesbit
jaesbit merged commit 826be4d into main Jul 4, 2026
@jaesbit
jaesbit deleted the feat/t51-ctrl-enter-start branch July 4, 2026 10:56
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