-
Notifications
You must be signed in to change notification settings - Fork 40
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
Handle terminal resize #7
Comments
Thanks @btwiuse for filing the issue. I am not using terminado for running Terminals. Which OS are you using ? Are you running Zasper as an electron app or web app ? |
I compiled it from source on macOS Sequoia 15.2 with latest master branch, started as a web app via
|
Default shell for mac should be |
Actually this bug was also reproducible on any linux server. I created a branch with .devcontainer.json, so you can directly view it in GitHub Codespace: Create a new workspace and wait ~5 minutes until Open the forwarded address, choose new terminal, open tmux in it, and resize browser window or the terminal You can see the inner tmux window doesn't resize |
As a side note I also changed the ui to use relative paths instead of hardcoded I believe the hardcoded hostname should be avoided in frontend ui. Maybe this deserves a separate issue |
Thanks for pointing it out. |
Thanks @btwiuse , I will look into tmux support. |
Is your feature request related to a problem? Please describe.
Currently zasper web terminal doesn't resize to fit the window
Describe the solution you'd like
Make the terminal backend handle resize events emitted from the xterm.js frontend
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
resize
commandAdditional context
The text was updated successfully, but these errors were encountered: