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

[BR] X-terminal does not change its dimensions by changing the panes dimension #124

Closed
2 of 3 tasks
aminya opened this issue Jul 25, 2020 · 11 comments · Fixed by #129
Closed
2 of 3 tasks

[BR] X-terminal does not change its dimensions by changing the panes dimension #124

aminya opened this issue Jul 25, 2020 · 11 comments · Fixed by #129
Labels
bug report 🐛 When something isn't working released 📮 Release has been made

Comments

@aminya
Copy link
Contributor

aminya commented Jul 25, 2020

  • [] I have reset Atom to defaults prior to submitting report.
  • I have not reset Atom to defaults prior to submitting report.
  • I have tried uninstalling/reinstalling the terminal.
  • I have not tried uninstalling/reinstalling the terminal.

Description

X-terminal does not change its dimensions by changing the panes dimension

Steps to reproduce

  1. Open a terminal and put in the bottom pane (besides) linter.
  2. Resize so it's big.
  3. type enter a couple of times so it gets filled.
  4. now make the pane smaller
  5. x-terminal does not resize like how Windows terminal resizes
    You can do this the other way around. Start with small and then make it bigger. The content does not go into the now empty black part.

Expected behavior:
Resizabliltiy like Windows terminal

Actual behavior:

I expect my pointer to be the last line, and I should not have to scroll for seeing it.
image

Reproduces how often:
Always

Versions

❯ atom -v

Atom    : 1.49.0       
Electron: 5.0.13       
Chrome  : 73.0.3683.121
Node    : 12.0.0       
❯ apm -v
apm  2.5.0
npm  6.14.5
node 10.20.1 x64    
atom 1.49.0
python 3.8.5        
git 2.27.0.windows.1
visual studio 2015  

OS name and version: Win 10
Terminal version: 8.2.3

Additional context

@aminya aminya added the bug report 🐛 Something isn't working label Jul 25, 2020
@UziTech
Copy link
Member

UziTech commented Jul 25, 2020

This is a known issue in xterm.js

@UziTech UziTech added xterm-bug 🐛 bug with a related issue in xterm.js and removed bug report 🐛 Something isn't working labels Jul 25, 2020
@UziTech
Copy link
Member

UziTech commented Jul 25, 2020

Maybe it changes based on the terminal but I think Powershell 7 works the way it is supposed to:

x-terminal-scroll
powershell-scroll

@UziTech
Copy link
Member

UziTech commented Jul 25, 2020

or are you referring to horizontal resizing?

it seems to work well for the current line:

x-terminal-resize

but this is what I meant is a known issue in xterm. There are only a certain number of lines they re-render on resize:

image

@UziTech
Copy link
Member

UziTech commented Jul 25, 2020

I wouldn't really consider this a bug but just a limitation of the software.

@aminya
Copy link
Contributor Author

aminya commented Jul 26, 2020

I usually use the terminal either on the right side or on the bottom depending on the work.

Yes, the side resizing does not work always.

The bottom resizing works sometimes, but not always. I can run clear once in a while to bring the cursor at the top. Maybe there is a command to bring it to the top without clearing (!?). Like this: https://github.com/JunoLab/atom-ink/blob/e151e3d629bce5f0235505a91fdd1ed2895d0ca9/lib/console/console.js#L290

x-terminal

@UziTech
Copy link
Member

UziTech commented Jul 26, 2020

So your problem is that x-terminal doesn't move the text up when the cursor goes below the bottom of the screen?

Can you share your x-terminal configuration and your powershell configuration?

@aminya
Copy link
Contributor Author

aminya commented Jul 26, 2020

So your problem is that x-terminal doesn't move the text up when the cursor goes below the bottom of the screen?

Can you share your x-terminal configuration and your powershell configuration?

Most of the things for x-terminal are on default
image
image

For PowerShell I use a couple of fancy things like fonts or git status here and there, but those do not affect the terminal itself.

image
image
image
image

@UziTech
Copy link
Member

UziTech commented Jul 26, 2020

do you have anything in "xterm.js Terminal Options" setting?

@UziTech UziTech added bug report 🐛 When something isn't working unable to duplicate !! When the issue cannot be duplicated and removed xterm-bug 🐛 bug with a related issue in xterm.js labels Jul 26, 2020
@aminya
Copy link
Contributor Author

aminya commented Jul 26, 2020

No. Everything is default:
image

@UziTech UziTech removed the unable to duplicate !! When the issue cannot be duplicated label Jul 27, 2020
@UziTech
Copy link
Member

UziTech commented Jul 27, 2020

I think I figured out what is happening. We call refitTerminal whenever the terminal is resized but when dragging to a different dock x-terminal thought the terminal was hidden so it wouldn't refit.

In #129 I removed the observer after it is initially visible so it stops marking it as invisible.

If you could, try the code from that PR and see if it fixes your issue.

@github-actions
Copy link

🎉 This issue has been resolved in version 9.0.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions bot added the released 📮 Release has been made label Jul 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report 🐛 When something isn't working released 📮 Release has been made
Development

Successfully merging a pull request may close this issue.

2 participants