Skip to content
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.

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

[Suggestion]: Keyboard navigation for Run and Clear #11

Open
tofflos opened this issue Oct 7, 2023 · 10 comments
Open

[Suggestion]: Keyboard navigation for Run and Clear #11

tofflos opened this issue Oct 7, 2023 · 10 comments
Assignees
Labels
enhancement New feature or request playground

Comments

@tofflos
Copy link

tofflos commented Oct 7, 2023

What happened?

I have to move my hands from the keyboard, grab the mouse, and click the Run and Clear buttons.
Afterwards I have to click the text area to continue editing.

Consider adding a keyboard shortcut such as Control + Return for launching Run and showing the keyboard shortcut as a tooltip.
Consider moving the keyboard focus back to text area after launching Run and Clear.

OS

Windows

What browsers are you seeing the problem on?

Firefox

Contact Details (optional)

No response

Additional information

Leave blank

@tofflos tofflos added bug Something isn't working playground labels Oct 7, 2023
@delabassee delabassee self-assigned this Oct 7, 2023
@delabassee
Copy link
Member

It's a bit difficult to find bindings that are relatively obvious and free.
I suggest Ctrl+Enter for "Run" and Ctrl+Backspace for "Clear", +Enter & +Backspace on Mac.

@nipafx
Copy link

nipafx commented Oct 9, 2023

Ctrl+Enter for "Run" is great but Ctrl+Backspace is already taken for "delete word before cursor" (in most text editors and browsers). You could use Ctrl+Shift+Backspace for that.

@delabassee
Copy link
Member

Not a big fan of a 3 keys combo. What about Ctrl+Esc instead?

@danthe1st
Copy link

Ctrl+Esc is at least used by Windows for bringing up the start menu.

@carimura
Copy link
Member

carimura commented Oct 9, 2023

+1 to command+return on Mac for running -- makes sense and is consistent with other behaviour (like sending this comment). I think sending the focus to the code window after run makes sense.

It's fine If we don't settle on clear for now as cmd+a then delete is a native way to clear the window without having to map anything and it keeps the focus.

@delabassee
Copy link
Member

Note that "clear" clears both the editor and the console.

@delabassee delabassee added enhancement New feature or request and removed bug Something isn't working labels Oct 9, 2023
@tofflos
Copy link
Author

tofflos commented Oct 9, 2023

I'd be super happy with just the Run shortcut and I'm not sure I would use a Clear shortcut often enough to warrant a shortcut.

@delabassee
Copy link
Member

Isn't Esc a good "Clear" (console & editor) shortcut?

@nipafx
Copy link

nipafx commented Oct 13, 2023

Ctrl+Shift+Backspace is not uncommon for larger deletes, e.g. in GitHub comment fields it deletes a line.

Isn't Esc a good "Clear" (console & editor) shortcut?

Only if a popup asks whether you really want to do that. Otherwise Esc is too commonly used to have such destructive effects.

@danthe1st
Copy link

I am unsure whether an option to clear the console together with the output is really necessary. After all, the console is cleared anyways when re-running the code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request playground
Projects
None yet
Development

No branches or pull requests

5 participants