-
Notifications
You must be signed in to change notification settings - Fork 213
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
Edit/Chat: Change shortcut and add dedicated setting #2865
Conversation
I went with the wording "Command Hints" but open to better ideas, cc @abeatrix @toolmantim |
"cody.commandHints.enabled": { | ||
"order": 12, | ||
"title": "Cody Command Hints", | ||
"type": "boolean", | ||
"markdownDescription": "Enable hints for Edit and Chat shortcuts, displayed alongside editor selections.", | ||
"default": false | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This being more visible means users are going to start enabling it. I personally think we should enable it by default once we have the logic to show custom keyboard shortcuts if users try to change it
02b406b
to
a99945f
Compare
## Description These shortcuts were accidentally not enabled everywhere as part of #2865 We can enable these shortcuts fully now, even when the command hint setting is off ## Test plan 1. Check the `unstable` flag is off 2. Check shortcuts show normally <!-- Required. See https://sourcegraph.com/docs/dev/background-information/testing_principles. -->
Description:
Test plan