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

improve inline chat menu #184943

Merged
merged 7 commits into from Jun 13, 2023
Merged

improve inline chat menu #184943

merged 7 commits into from Jun 13, 2023

Conversation

meganrogge
Copy link
Contributor

Tyriar
Tyriar previously requested changes Jun 12, 2023
content.push(localize('inlineChat.overview', "Inline chat occurs within a code editor and takes into account current selection. It is useful for refactoring, fixing, and more. Keep in mind that Copilot generated code may be incorrect."));
content.push(localize('inlineChat.access', "It can be activated via the Fix and Explain with Copilot context menu actions or directly using the command: Inline Chat: Start Code Chat {0}.", startChatKeybininding));
content.push(localize('chat.requestHistoryInline', 'In the input box, use UpArrow/DownArrow to navigate your request history. Edit input and use enter or the make request button to run a new request.'));
content.push(localize('inlineChat.contextActions', "Explain and Fix with Copilot actions run a request prefixed with /fix or /explain. These prefixes can be used directly in the input box to apply those specific actions."));
Copy link
Member

Choose a reason for hiding this comment

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

I don't think "Copilot" is meant to be mentioned in core.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

These are what i'm describing - have to describe them because they're very discoverable

Screenshot 2023-06-12 at 4 06 27 PM

meganrogge and others added 5 commits June 12, 2023 16:04
…yHelp.ts

Co-authored-by: Daniel Imms <2193314+Tyriar@users.noreply.github.com>
…yHelp.ts

Co-authored-by: Daniel Imms <2193314+Tyriar@users.noreply.github.com>
…yHelp.ts

Co-authored-by: Daniel Imms <2193314+Tyriar@users.noreply.github.com>
@meganrogge meganrogge disabled auto-merge June 12, 2023 22:15
@meganrogge meganrogge closed this Jun 12, 2023
auto-merge was automatically disabled June 12, 2023 22:15

Pull request was closed

@meganrogge meganrogge reopened this Jun 12, 2023
@meganrogge meganrogge disabled auto-merge June 13, 2023 00:03
@meganrogge meganrogge merged commit 74d1f59 into main Jun 13, 2023
9 checks passed
@meganrogge meganrogge deleted the merogge/inline-help-improvement branch June 13, 2023 00:12
@github-actions github-actions bot locked and limited conversation to collaborators Jul 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants