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

Persistant terminal session on tab refresh #2276

Closed
bpmct opened this issue Nov 6, 2020 · 11 comments · Fixed by #3308
Closed

Persistant terminal session on tab refresh #2276

bpmct opened this issue Nov 6, 2020 · 11 comments · Fixed by #3308
Labels
feature New user visible feature
Milestone

Comments

@bpmct
Copy link
Member

bpmct commented Nov 6, 2020

Eclipse Theia keeps any terminal sessions open when the tab is refreshed. Would it be possible to have this same functionality in code-server? Either enabled at all times, or with a flag.

@bpmct bpmct added the feature label Nov 6, 2020
@ntimo
Copy link

ntimo commented Nov 9, 2020

VSCode 1.51 has been released with support for "Ability to persist/reconnect to terminal sessions." might this actually fix this issue?

Source: https://code.visualstudio.com/updates/v1_51

@code-asher
Copy link
Member

code-asher commented Nov 9, 2020 via email

@code-asher code-asher self-assigned this Nov 9, 2020
@ntimo
Copy link

ntimo commented Nov 20, 2020

I just installed code server v3.7.2 which contains the update to vscode 1.51 but sadly the terminal sessions are not persistent :(

@code-asher
Copy link
Member

code-asher commented Nov 20, 2020 via email

@nhooyr nhooyr added enhancement Some improvement that isn't a feature and removed feature labels Dec 7, 2020
@nhooyr nhooyr added feature New user visible feature and removed enhancement Some improvement that isn't a feature labels Dec 8, 2020
@outermeasure
Copy link
Contributor

Any news on this? @code-asher Do you have a starting point so I can take a look?

@code-asher
Copy link
Member

code-asher commented Feb 12, 2021 via email

@outermeasure
Copy link
Contributor

Thank you. I will send a PR if I get anywhere (but no promises).

@outermeasure
Copy link
Contributor

Ok, I found a related but different problem that was a bit more irritating. Terminal processes leak when refreshing the page.
I have a PR for that: #2723

@ntimo
Copy link

ntimo commented Mar 25, 2021

@outermeasure @code-asher Do you have any news about this? :) This would be super important since for example when your run Ansible from Code Server and your connection for what ever reason breaks being able to reconnect and still having the open terminal would be quite nice.

@code-asher
Copy link
Member

Nope, we haven't done anything here yet.

@jsjoeio jsjoeio added this to the Backlog milestone Apr 29, 2021
code-asher added a commit to code-asher/code-server that referenced this issue May 7, 2021
Most of this was a straightforward replacement of our code with theirs
but I also removed `getDefaultShellAndArgs` since it seems the reference
implementation no longer does that either.

Fixes coder#2276.
@code-asher code-asher modified the milestones: Backlog, v3.9.4 May 7, 2021
code-asher added a commit to code-asher/code-server that referenced this issue May 7, 2021
Most of this was a straightforward replacement of our code with theirs
but I also removed `getDefaultShellAndArgs` since it seems the reference
implementation no longer does that either.

Fixes coder#2276.
@raychunghk
Copy link

Now i use control+shift+p , then "Terminal: Attach to Session" to resume the terminal session I was working on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New user visible feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants