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

A tab might not close #145685

Closed
bpasero opened this issue Mar 22, 2022 · 3 comments
Closed

A tab might not close #145685

bpasero opened this issue Mar 22, 2022 · 3 comments
Assignees
Labels
api insiders-released Patch has been released in VS Code Insiders polish Cleanup and polish issue workbench-tabs VS Code editor tab issues
Milestone

Comments

@bpasero
Copy link
Member

bpasero commented Mar 22, 2022

Testing #145585

Closes the tab. This makes the tab object invalid and the tab should no longer be used for further actions.

Not if the tab is dirty and the user cancels the dialog.

@lramos15
Copy link
Member

Maybe the promise should return an array of the closed tabs or something along those lines. How will the user know which tabs are still good? How do we know what tabs are cancelled from group.CloseEditors?

@bpasero
Copy link
Member Author

bpasero commented Mar 22, 2022

Workbench tends to return a boolean, but interestingly not for closeEditors, only closeEditor.

@lramos15
Copy link
Member

Workbench tends to return a boolean, but interestingly not for closeEditors

Wonder if we should enhance it to turn an array of booleans that way we know which closes were successful and which were not

@lramos15 lramos15 added api polish Cleanup and polish issue workbench-tabs VS Code editor tab issues labels Mar 23, 2022
@lramos15 lramos15 added this to the March 2022 milestone Mar 23, 2022
@github-actions github-actions bot locked and limited conversation to collaborators May 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api insiders-released Patch has been released in VS Code Insiders polish Cleanup and polish issue workbench-tabs VS Code editor tab issues
Projects
None yet
Development

No branches or pull requests

3 participants
@bpasero @lramos15 and others