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

Hitting escape when editing a comment should confirm cancellation. #1455

Open
rwspeight-orange opened this issue Nov 27, 2019 · 2 comments · May be fixed by microsoft/vscode#236589
Open

Hitting escape when editing a comment should confirm cancellation. #1455

rwspeight-orange opened this issue Nov 27, 2019 · 2 comments · May be fixed by microsoft/vscode#236589
Assignees
Labels
comments feature-request Request for new features or functionality upstream/vscode
Milestone

Comments

@rwspeight-orange
Copy link

Hitting escape while editing a comment terminates edit mode and abandons unsaved work. Confirming the cancellation with the user would help prevent lost work and user frustration.

@rwspeight-orange rwspeight-orange changed the title Hitting escape when editing a comment should confirm cancellation of comment. Hitting escape when editing a comment should confirm cancellation. Nov 27, 2019
@RMacfarlane RMacfarlane added comments feature-request Request for new features or functionality labels Dec 2, 2019
@wcerfgba
Copy link

Yes please, this is easily the most frustrating problem I have with the extension at present. I can't tell you how many times I've accidentally knocked Esc and lost a PR comment with no way to recover it 😭

@RikkiGibson
Copy link
Member

It looks like this issue is still occurring. For comparison, if you hit esc while drafting a PR comment on the GitHub website, nothing happens.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comments feature-request Request for new features or functionality upstream/vscode
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants