Skip to content

feat: add 'q' shortcut to create SQL cell in command mode - #10755

Open
mahesh-desu wants to merge 1 commit into
marimo-team:mainfrom
mahesh-desu:feature/q-sql-cell-shortcut
Open

feat: add 'q' shortcut to create SQL cell in command mode#10755
mahesh-desu wants to merge 1 commit into
marimo-team:mainfrom
mahesh-desu:feature/q-sql-cell-shortcut

Conversation

@mahesh-desu

Copy link
Copy Markdown

Fixes #10688

Add command.createCellAfterSQL hotkey (key: q) that creates a new cell below and switches it to SQL in one step.

What this does:

  • Press Esc then q in command mode to create a new SQL cell below the current cell
  • Previously, you had to create a Python cell with �, then toggle it to SQL with Alt+Shift+L

Changes (3 files, +72 lines):

  • rontend/src/core/hotkeys/hotkeys.ts: Add q binding for command.createCellAfterSQL
  • rontend/src/components/editor/navigation/navigation.ts: Handler creates cell, ensures editor mount via ensureCellEditorView, switches language to SQL
  • rontend/src/components/editor/navigation/tests/navigation.test.ts: Test verifying cell creation + SQL switch

Verified locally:

  • 71/71 tests pass
  • Lint: 0 warnings, 0 errors
  • Format: correct

I have read the CLA Document and I hereby sign the CLA

@vercel

vercel Bot commented Sep 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
marimo-docs Ready Ready Preview Sep 6, 2026 4:47am UTC

Request Review

@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@mahesh-desu

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

Add command.createCellAfterSQL hotkey (key: 'q') that creates a new
cell below and switches it to SQL in one step, matching the workflow
described in issue marimo-team#10688.

Changes:
- hotkeys.ts: Add 'q' binding for command.createCellAfterSQL
- navigation.ts: Handler creates cell, ensures editor mount, switches language
- navigation.test.ts: Test verifying cell creation + SQL switch

Closes marimo-team#10688
@mahesh-desu

Copy link
Copy Markdown
Author

recheck

1 similar comment
@mahesh-desu

Copy link
Copy Markdown
Author

recheck

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.

Add a command-mode shortcut to create a SQL cell below

1 participant