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

Add a dialog box when opening unsaved files in a custom editor #114947

Merged
merged 2 commits into from
Feb 2, 2021
Merged

Add a dialog box when opening unsaved files in a custom editor #114947

merged 2 commits into from
Feb 2, 2021

Conversation

lramos15
Copy link
Member

Currently when triggering the reopen with command on files with unsaved channels leads to the custom editor being opened, but the existing tab staying open. This PR prompts the user to save or discard their changes before the custom editor is opened.

image

@lramos15 lramos15 requested a review from mjbvz January 25, 2021 19:40
@lramos15
Copy link
Member Author

@mjbvz Ideally, as discussed, we would like to make this work for notebooks. I'm trying to identify the best place to move the code but the open flows seem to diverge around this point https://github.com/microsoft/vscode/blob/master/src/vs/workbench/services/editor/common/editorOpenWith.ts#L111. I was wondering if you had any thoughts on to where the fix would live to cover both places

@lramos15 lramos15 merged commit 33f49b2 into microsoft:master Feb 2, 2021
lramos15 added a commit that referenced this pull request Feb 2, 2021
lramos15 added a commit that referenced this pull request Feb 2, 2021
MarcoZehe pushed a commit to MarcoZehe/vscode that referenced this pull request Feb 3, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Mar 19, 2021
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.

Hex Editor: original untitled file editor is not closed when opening in the Hex Editor
2 participants