You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Make "copy line down" hotkey in the shortcuts page of the marimo editor editable #7056
I would like to ask to make the "copy line(s) down" hotkey in the shortcuts page of the marimo editor editable, if feasible (possible also the "copy line(s) up" hotkey). Currently, the hotkey is not editable because #6843 didn't override it in the codemirror settings. The reason is that I would find it nice if I could switch to Ctrl + D.
I’ve already started working on this issue and would like to be officially assigned.
Please assign me so I can continue with the implementation and submit a PR. Thanks!
Hi — checking in on this one, as it looks unclaimed (#6843 landed the panel entries, but left both hotkeys editable: false).
I have a fix working locally that follows the override pattern already used for toggleComment/toggleBlockComment in frontend/src/core/codemirror/keymaps/keymaps.ts:
add copyLineUp/copyLineDown to OVERRIDDEN_COMMANDS, so CodeMirror's hardcoded Shift-Alt-Arrow bindings are filtered out of the default keymap
re-bind them in overrideKeymap() using keymap.getHotkey(...), so they follow the user's configured key
drop editable: false from both entries in hotkeys.ts
Default keybindings are unchanged; only the ability to reconfigure them is added. make fe-check passes and the existing generic assertions in keymaps.test.ts (which compare overrideKeymap().length against OVERRIDDEN_COMMANDS.size) pick up the new commands automatically. I've added a test asserting an override to Ctrl-D actually reaches the keymap — the rebinding @mullimanko originally asked for.
I'll open this as a draft PR shortly so the diff is easy to look at. Since this issue doesn't carry the ready label, please treat the PR as awaiting your go-ahead rather than a review request — happy to close it if you'd rather this waited or went a different way.
Disclosure: this change was authored by a coding agent, reviewed by me before posting.
Activity
kallal79 commentedon Apr 3, 2026
I’ve already started working on this issue and would like to be officially assigned.
Please assign me so I can continue with the implementation and submit a PR. Thanks!
Abdulrehman-PIAIC80387 commentedon Jul 20, 2026
Hi — checking in on this one, as it looks unclaimed (#6843 landed the panel entries, but left both hotkeys
editable: false).I have a fix working locally that follows the override pattern already used for
toggleComment/toggleBlockCommentinfrontend/src/core/codemirror/keymaps/keymaps.ts:copyLineUp/copyLineDowntoOVERRIDDEN_COMMANDS, so CodeMirror's hardcodedShift-Alt-Arrowbindings are filtered out of the default keymapoverrideKeymap()usingkeymap.getHotkey(...), so they follow the user's configured keyeditable: falsefrom both entries inhotkeys.tsDefault keybindings are unchanged; only the ability to reconfigure them is added.
make fe-checkpasses and the existing generic assertions inkeymaps.test.ts(which compareoverrideKeymap().lengthagainstOVERRIDDEN_COMMANDS.size) pick up the new commands automatically. I've added a test asserting an override toCtrl-Dactually reaches the keymap — the rebinding @mullimanko originally asked for.I'll open this as a draft PR shortly so the diff is easy to look at. Since this issue doesn't carry the
readylabel, please treat the PR as awaiting your go-ahead rather than a review request — happy to close it if you'd rather this waited or went a different way.Disclosure: this change was authored by a coding agent, reviewed by me before posting.
fix(hotkeys): make copy line up/down hotkeys editable