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

Horizontal resize screws up horizontal terminal layout #8336

Closed
isidorn opened this issue Jun 28, 2016 · 6 comments
Closed

Horizontal resize screws up horizontal terminal layout #8336

isidorn opened this issue Jun 28, 2016 · 6 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug terminal Integrated terminal issues upstream Issue identified as 'upstream' component related (exists outside of VS Code) verified Verification succeeded
Milestone

Comments

@isidorn
Copy link
Contributor

isidorn commented Jun 28, 2016

#8222

  1. Open integrated terminal - run some scripts which uses all the horizontal space
  2. Resize vscode window horizontaly - notice how the terminal will shrink but will never re-grow
  3. I would expect the terminal grows horizontally as do the other terminals

terminalresize

@isidorn isidorn added the terminal Integrated terminal issues label Jun 28, 2016
@Tyriar
Copy link
Member

Tyriar commented Jun 28, 2016

Upstream issue for prompt messing up on resize: xtermjs/xterm.js#103
Upstream issue for output messing up on resize: xtermjs/xterm.js#325

@Tyriar
Copy link
Member

Tyriar commented Nov 28, 2016

I've consolidated all issues related to horizontally resizing the viewport under this one. It's important to note that Windows seems to behave a little worse when resizing than other platforms, wrapping some lines incorrectly. These are still all upstream issues as mentioned in #8336 (comment)

@lrs
Copy link

lrs commented Jan 9, 2017

Several months later and no update? What does "upstream" mean in this context by the way?

I only add a comment because the integrated terminal feature seems to be a major factor when considering whether to go with Code or not.

Is this a problem with Windows installations only? I can use and resize an external cmd window just fine when I exec npm run build that is watching for changes for example.

@Tyriar
Copy link
Member

Tyriar commented Jan 9, 2017

Upstream means xterm.js. There is a community member who has a PR out at the moment but it needs to be reworked after some big performance related improvements. It applies to all operating systems.

@lrs
Copy link

lrs commented Jan 9, 2017

OK, thanks for the update.

@Tyriar
Copy link
Member

Tyriar commented Mar 30, 2017

The resize discarding data is fixed in Insiders, the data will now reappear when resized back thanks xtermjs/xterm.js#616 from @LucianBuzzo.

@LucianBuzzo also has a PR out that isn't merged just yet that adds support for reflowing the lines so they will get wrapped when the horizontal size is reduced! This is tracked in #23688

@Tyriar Tyriar closed this as completed Mar 30, 2017
@Tyriar Tyriar modified the milestones: March 2017, Backlog Mar 30, 2017
@chrmarti chrmarti added the verified Verification succeeded label Mar 30, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
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 terminal Integrated terminal issues upstream Issue identified as 'upstream' component related (exists outside of VS Code) verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants