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

Allow resizing on attached pipes too. #1979

Merged
merged 1 commit into from
Nov 4, 2016
Merged

Conversation

foot
Copy link
Contributor

@foot foot commented Nov 4, 2016

Fixes #1977

@errordeveloper
Copy link
Contributor

It works in general, i.e. the terminal uses full-width:

And resizing affects environment variables:

However, having resized to a very large size, and making it smaller again leads to shell prompt way below the last visible line. I'm not sure if this is something we may be able to fix, or it's a more general issue, I still don't fully grasp how exactly this stuff works.

@errordeveloper
Copy link
Contributor

errordeveloper commented Nov 4, 2016

However, having resized to a very large size, and making it smaller again leads to shell prompt way below the last visible line. I'm not sure if this is something we may be able to fix, or it's a more general issue, I still don't fully grasp how exactly this stuff works.

Having observed it my initial test, I am not able to reproduce it now... I think this may be a separate issue to do some kind of edge-case. I'll report if I see it again, this PR LGTM 👍

@2opremio
Copy link
Contributor

2opremio commented Nov 4, 2016

@errordeveloper The exec terminals (>_) are not relevant for this PR. Did you test the logs (attach)?

@2opremio
Copy link
Contributor

2opremio commented Nov 4, 2016

@foot It would be cool if the control icon could be placed on a top corner of the terminal window (to know what control triggered it)

@davkal
Copy link
Contributor

davkal commented Nov 4, 2016

LGTM

@davkal davkal merged commit 722d5cd into master Nov 4, 2016
@davkal davkal deleted the 1977-fix-notty-resizing branch November 4, 2016 12:30
@errordeveloper
Copy link
Contributor

errordeveloper commented Nov 4, 2016

I've retested with logs.

Here is first one, looks good.

However, making the window wider doesn't reflow the log lines.

But closing and re-opening works ok.

Making the window smaller and larger again leave empty space...

@2opremio
Copy link
Contributor

2opremio commented Nov 4, 2016

Thanks @errordeveloper, better late than never. Can you please create an issue with this?

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

Successfully merging this pull request may close these issues.

4 participants