forcedotcom / Einstein-GPT-for-Developers

Einstein GPT For Developers
BSD 3-Clause "New" or "Revised" License
31 stars 8 forks source link

Feedback regarding keyboard Shortcuts #27

Closed tahabasri closed 9 months ago

tahabasri commented 11 months ago

Originally posted in salesforcedx-vscode-einstein-gpt.

The default provided shortcuts of the extension are overriding some of the most important system shortcuts in VSCode, specially for Mac users. I think this issue can be discussed/resolved as the list of shortcuts is still small and maintainable.

How to reproduce ? MacOS only

Expected ?

Cursor should go to end of line

Current behavior

Cursor is stuck on the initial position.

Why ?

The accept command trigger (cmd+right) overrides the System shortcut cursorEnd, the latter is the commonly used within VSCode. User must either change system settings after installing the extension which is not ideal (given that this change applies to Apex files only).

image

Another useful shortcut that's impacted is the famous cmd+r which is now triggering Einstein: Generate Code instead of reloading VSCode by default.

image

Suggestions ?

My suggestion is to go with cmd+shift+key as this applies for a small set of File types (Apex for now), the learning curve is not much impacted as the keyset is still small. I'm open for any other type of suggestions. Thanks.

tahabasri commented 11 months ago

As I can't push code to the original repo, here are my suggested changes: image

floralan commented 10 months ago

Hi @tahabasri, our team is aware of this issue. We have created an internal work item (W-14194362) to fix it. Thank you for your input!

d-gunawan commented 9 months ago

@tahabasri, we changed our default keybindings for acceptSuggestion and promptForQuery in version 0.23.1 so they should no longer conflict with the system keybindings, more details can be found in our release notes.

The insertCommentForCodeGeneration feature is still behind a feature flag, so it shouldn't cause any keybinding conflict.

tahabasri commented 9 months ago

Great ! Thanks @d-gunawan !