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

[sidebar]Double click new file make all files display grey #47606

Closed
oDoraemon opened this issue Apr 10, 2018 · 3 comments
Closed

[sidebar]Double click new file make all files display grey #47606

oDoraemon opened this issue Apr 10, 2018 · 3 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug file-explorer Explorer widget issues verified Verification succeeded
Milestone

Comments

@oDoraemon
Copy link

Issue Type: Bug

Double Click new file icon in sidebar get all files grey

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

Extensions (4)
Extension Author (truncated) Version
vscode-eslint dba 1.4.8
php-intellisense fel 2.3.1
php-pack fel 1.0.1
python ms- 2018.3.1
@joaomoreno
Copy link
Member

joaomoreno commented Apr 11, 2018

@isidorn Setting candidate for discussion.

@joaomoreno joaomoreno added the candidate Issue identified as probable candidate for fixing in the next release label Apr 11, 2018
@vscodebot vscodebot bot removed the new release label Apr 12, 2018
@isidorn
Copy link
Contributor

isidorn commented Apr 12, 2018

I can reproduce it and it is a regression.
However the issue is of cosmetic nature the explroer is gray and no other behavior is broken. Also the bad behavior goes away if a user clicks on a new file icon again or if he restarts vscode.
Due to the above I belive this is not a candidate. @joaomoreno @bpasero let me know if you disagree and I can look into fixing it for the recovery

@isidorn isidorn removed the candidate Issue identified as probable candidate for fixing in the next release label Apr 12, 2018
@isidorn isidorn added this to the April 2018 milestone Apr 12, 2018
@isidorn isidorn added bug Issue identified by VS Code Team member as probable bug file-explorer Explorer widget issues labels Apr 12, 2018
@isidorn
Copy link
Contributor

isidorn commented Apr 19, 2018

This got introduced because we switched to use a map instead of an array for children.
So the NewStatPlaceholder would get ovewritten in the map.
Fix: made the new file / new folder actions noop when the parent is in the process of creating a new entry

@tsalinger tsalinger added the verified Verification succeeded label Apr 26, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Jun 3, 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 file-explorer Explorer widget issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants