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

Multiple "do you want to save changes" dialogs #50539

Closed
d07RiV opened this issue May 28, 2018 · 5 comments
Closed

Multiple "do you want to save changes" dialogs #50539

d07RiV opened this issue May 28, 2018 · 5 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded workbench-editors Managing of editor widgets in workbench window
Milestone

Comments

@d07RiV
Copy link

d07RiV commented May 28, 2018

Issue Type: Bug

  1. Open multiple files in different tabs (optional)
  2. Modify one of the files
  3. Switch to a different, preferably far away tab (optional)
  4. Press and hold Ctrl+F4, attempting to close all tabs
  5. When VS code gets to your modified file, it emits multiple "do you want to save changes" dialogs, which have to be resolved one by one, because of queued ctrl+f4 inputs.

Expected behavior: only one dialog should be displayed.

(I'm aware of Ctrl+K/W, which does avoid this issue)

VS Code version: Code 1.22.1 (950b8b0, 2018-04-06T02:26:57.615Z)
OS version: Windows_NT x64 10.0.17134

System Info
Item Value
CPUs Intel(R) Core(TM) i7-6700 CPU @ 3.40GHz (8 x 3408)
Memory (System) 15.95GB (3.97GB free)
Process Argv C:\Program Files\Microsoft VS Code\Code.exe
Screen Reader no
VM 0%
Extensions (10)
Extension Author (truncated) Version
vscode-eslint dba 1.4.10
prettier-vscode esb 1.3.1
docthis joe 0.6.0
sublime-babel-vscode jos 0.2.10
cpptools ms- 0.17.3
node-debug2 ms- 1.22.7
debugger-for-chrome msj 4.4.3
shader sle 1.1.3
autoimport ste 1.5.3
vscode-react-native vsm 0.6.10
@bpasero
Copy link
Member

bpasero commented May 28, 2018

@d07RiV does it emit multiple dialogs for the same file or different files?

@bpasero bpasero added the info-needed Issue requires more information from poster label May 28, 2018
@d07RiV
Copy link
Author

d07RiV commented May 29, 2018

For the same file. I click save on one, but it still asks me whether I want to save it or not.

@bpasero
Copy link
Member

bpasero commented May 29, 2018

/findDuplicates

@bpasero bpasero added bug Issue identified by VS Code Team member as probable bug workbench-editors Managing of editor widgets in workbench window and removed info-needed Issue requires more information from poster labels May 29, 2018
@bpasero bpasero added this to the June 2018 milestone May 29, 2018
@bpasero
Copy link
Member

bpasero commented May 29, 2018

I can reproduce, thinking of maybe introducing some code that will check if the request for saving an editor is running and then joining that request to prevent duplication.

Looks like the fix from 590a611 no longer works.

@bpasero bpasero closed this as completed Jun 2, 2018
@roblourens roblourens added the verified Verification succeeded label Jun 28, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Jul 17, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded workbench-editors Managing of editor widgets in workbench window
Projects
None yet
Development

No branches or pull requests

3 participants