Skip to content

add duplicate selection shortcut to hotkeys as part of editing command #6843

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Oct 23, 2025
Merged

add duplicate selection shortcut to hotkeys as part of editing command #6843

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
d577582
add duplicate selection shortcut to hotkeys as part of editing command
minsun-ss Oct 20, 2025
f886b5d
Update frontend/src/core/hotkeys/hotkeys.ts
minsun-ss Oct 20, 2025
e4e8ec4
fix: adjust typo on the keyboard shortcut
minsun-ss Oct 20, 2025
8fc134b
feat: renamed item to copy line(s) down to match codemirror settings and
minsun-ss Oct 23, 2025
7291c90
nit: changed the cell commands to match codemirror names and instead
minsun-ss Oct 23, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions frontend/src/core/hotkeys/hotkeys.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -239,6 +239,11 @@ const DEFAULT_HOT_KEY = {
group: "Editing",
key: "F2",
},
"cell.duplicateSelection": {
name: "Duplicate selection",
group: "Editing",
key: "Ctrl-Shift-ArrowDown",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
key: "Ctrl-Shift-ArrowDown",
key: "Ctrl-Shift-ArrowDown",
editable: false,

since right now we don't override it in the codemirror settings, this needs to be set to not editable

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apologies about this - missed the codemirror related items. I do see that you use the up/down shortcuts as copy line up/down, I can convert mine to reflect that nomenclature for consistency, if that's desired?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For me, it copies full lines. so i think Copy line(s) down would be fine text.
I would also maybe add Copy line(s) up as well

Copy link
Contributor Author

@minsun-ss minsun-ss Oct 23, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That makes sense - added the two lines and set them both to noneditable.

image

},

// Markdown
"markdown.bold": {
Expand Down