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

Terminal: remember layout between restarts #109244

Closed
bpasero opened this issue Oct 23, 2020 · 3 comments
Closed

Terminal: remember layout between restarts #109244

bpasero opened this issue Oct 23, 2020 · 3 comments
Assignees
Labels
feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders on-testplan terminal Integrated terminal issues
Milestone

Comments

@bpasero
Copy link
Member

bpasero commented Oct 23, 2020

Now that we support to restore terminals after connecting back to a remote, I think the terminal layout should also be preserved.

I would have expected that the terminal already stores the layout into the view memento, but it does not seem to be the case. I am not sure why?

//cc @alexdima

@bpasero bpasero added terminal Integrated terminal issues remote Remote system operations issues labels Oct 23, 2020
@alexr00 alexr00 removed their assignment Oct 26, 2020
@Tyriar Tyriar added feature-request Request for new features or functionality and removed remote Remote system operations issues labels Oct 30, 2020
@Tyriar Tyriar added this to the Backlog milestone Oct 30, 2020
@Tyriar
Copy link
Member

Tyriar commented Nov 9, 2020

Currently when remotes reconnect terminals the split terminals get flattened into their own terminals. This is mainly an issue of how we want to persist that knowledge across sessions and make sure the remote agent stores it (eg. splitIndex: number may be enough). Note also that split terminals are naturally smaller so they have non-standard dimensions, not sure we need to do anything special to handle dimensions differently as resize events are recorded.

Ideally there may need to be some logic/checks to ensure split terminals get adopted together so they cannot be split up. This will probably involve the introduction of the terminal tab idea on the server.

@GitMensch
Copy link
Contributor

Is this issue only related to the remote terminal or also to the way hidden terminals are processed (a terminal created by openssh with reveal:always has a correct size (full-width of the terminal view) while reveal:never seems to be 80 columns only)?

@roblourens
Copy link
Member

Not exactly sure what your scenario is, but I don't think this will be related.

@github-actions github-actions bot locked and limited conversation to collaborators Jan 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders on-testplan terminal Integrated terminal issues
Projects
None yet
Development

No branches or pull requests

7 participants
@roblourens @bpasero @Tyriar @GitMensch @meganrogge @alexr00 and others