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

feat: add keyboard shortcuts for Editor actions #542

Merged
merged 1 commit into from
May 10, 2024

Conversation

PhilKes
Copy link
Contributor

@PhilKes PhilKes commented May 10, 2024

This PR closes #314

Adds Action to show CodeGPT Actions popup with keyboard shortcut:
popup-shortcuts

All Actions are assigned an alpha-numeric letter for seamless keyboard controls.

I didn't really know how to define the best default shortcut, so I sticked with the CTRL+SHIFT+ALT combo plus "M" (maybe "M" for "Menu"?), that doesn't seem to be used by anything too important:
image

<action id="CodeGPT.TriggerEditorPopup"
class="ee.carlrobert.codegpt.actions.editor.ShowEditorActionGroupAction"
text="Show CodeGPT Actions">
<keyboard-shortcut first-keystroke="ctrl shift alt m" keymap="$default"/>
Copy link
Owner

@carlrobertoh carlrobertoh May 10, 2024

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, as a Linux and Windows User I always forget about that "other" keyboard 😆
Aren't they missing for the toggle Code Completions Action too?

<keyboard-shortcut first-keystroke="ctrl shift alt c" keymap="$default"/>

<keyboard-shortcut first-keystroke="ctrl shift alt c" keymap="$default"/>

Copy link
Owner

Choose a reason for hiding this comment

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

Please ignore my last comment, sorry. The ctrl shift alt m combination is already valid on mac as shift option meta m. No changes needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I removed the additional keyboard-shortcuts again

@carlrobertoh carlrobertoh merged commit fcd0808 into carlrobertoh:master May 10, 2024
2 checks passed
@lyrl
Copy link

lyrl commented May 11, 2024

thank you very much for your hard work @PhilKes

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.

Feature Request: Implement Shortcut Keys for Directory Level Navigation
3 participants