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

Cancelling opening a folder from a multi-root unsaved workspace corrupts the current worksapce #42799

Closed
jtanx opened this issue Feb 2, 2018 · 3 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded workbench-multiroot Multi-root (multiple folders) issues
Milestone

Comments

@jtanx
Copy link

jtanx commented Feb 2, 2018

  • VSCode Version: 1.19.3
  • OS Version: Windows 10 10.0.16299.125

Steps to Reproduce:

  1. Have an unsaved multi-root workspace (e.g. add two folders to the same workspace)
  2. Have an integrated terminal open
  3. Attempt to open a folder (CTRL + K O)
  4. At the confirmation box, click cancel:
    image
  5. Terminal is now broken:
    image
  6. Various other things are broken. For example, I can't run extension commands any more.

Does this issue occur when all extensions are disabled?: Yes

@bpasero
Copy link
Member

bpasero commented Feb 2, 2018

Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/

@Tyriar you might be disposing something on the wrong phase (before shutdown vs on shutdown)?

@bpasero bpasero added the info-needed Issue requires more information from poster label Feb 2, 2018
@bpasero bpasero assigned Tyriar and unassigned bpasero Feb 2, 2018
@bpasero bpasero added the terminal Integrated terminal issues label Feb 2, 2018
@jtanx
Copy link
Author

jtanx commented Feb 2, 2018

I just tried on insiders and it does the same thing.

image

It's not just limited to the terminal, it's like it reaches a state where things are semi-destructed - including interacting with extensions - try run any extension command and it won't work.

Oh yeah, I've also noticed that the tab bar goes into a weird state as well - it no longer reflects the opened file. But this only happens sometimes; I find it harder to repro:

Click to expand

vscode

@bpasero bpasero assigned bpasero and unassigned Tyriar Feb 2, 2018
@bpasero bpasero added bug Issue identified by VS Code Team member as probable bug and removed terminal Integrated terminal issues info-needed Issue requires more information from poster labels Feb 2, 2018
@bpasero bpasero modified the milestones: On Deck, February 2018 Feb 2, 2018
@bpasero
Copy link
Member

bpasero commented Feb 2, 2018

I can reproduce. Should probably first ask the main window to veto the unload before even forwarding this to the window.

@bpasero bpasero added the workbench-multiroot Multi-root (multiple folders) issues label Feb 2, 2018
bpasero added a commit that referenced this issue Feb 5, 2018
@bpasero bpasero closed this as completed Feb 5, 2018
bpasero added a commit that referenced this issue Feb 5, 2018
@RMacfarlane RMacfarlane added the verified Verification succeeded label Mar 1, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Mar 22, 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-multiroot Multi-root (multiple folders) issues
Projects
None yet
Development

No branches or pull requests

4 participants