Skip to content

Way to "close" terminal session with "close folder" #3947

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

Closed
geiseri opened this issue Aug 10, 2021 · 3 comments
Closed

Way to "close" terminal session with "close folder" #3947

geiseri opened this issue Aug 10, 2021 · 3 comments
Labels
enhancement Some improvement that isn't a feature
Milestone

Comments

@geiseri
Copy link
Contributor

geiseri commented Aug 10, 2021

OS/Web Information

  • code-server: v3.11.1
  • VS Code: v1.57.1
  • Commit: c680aae
  • Date: 2021-08-06T18:33:37Z
  • Browser: Mozilla/5.0 (X11; CrOS x86_64 13982.69.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/92.0.4515.130 - Safari/537.36

Steps to Reproduce

  1. Open folder from the menu
  2. Mess around with files, terminals stuff...
  3. Close folder from the menu

Expected

All state is cleared from the project. Terminal ptys released, extensions unloaded, etc.

Actual

"Session" is still running:
image

Notes

It used to be that state was lost on close, so the current behavior is orders of magnitude more usable. I think I am seeing a strangeness related to PR #2723. Code server is a bit different from VS Code in that there is a difference between the window being closed, and the folder is closed. Since it's a server it could be the browser crashed. Would it be possible to tease those apart? That being said, if that is too invasive I can live with needing to close the shell manually vs having my terminal lost when Amtrak's wifi flakes out. =)

This issue can be reproduced in VS Code: No

@jsjoeio jsjoeio added the needs-investigation This issue needs to be further investigated label Aug 10, 2021
@jsjoeio
Copy link
Contributor

jsjoeio commented Aug 10, 2021

cc @code-asher who is more familiar with this part of code-server

@code-asher
Copy link
Member

code-asher commented Aug 12, 2021 via email

@code-asher code-asher added enhancement Some improvement that isn't a feature and removed needs-investigation This issue needs to be further investigated labels Aug 12, 2021
@code-asher code-asher added this to the Backlog milestone Oct 27, 2021
@stale
Copy link

stale bot commented Apr 25, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no activity occurs in the next 5 days.

@stale stale bot added the stale label Apr 25, 2022
@stale stale bot closed this as completed May 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Some improvement that isn't a feature
Projects
None yet
Development

No branches or pull requests

3 participants