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

[Enhancement] Increase ssh window size #587

Open
markovchainz opened this issue Dec 22, 2020 · 8 comments
Open

[Enhancement] Increase ssh window size #587

markovchainz opened this issue Dec 22, 2020 · 8 comments
Labels
help-wanted Extra attention is needed kind/feature New feature or request
Milestone

Comments

@markovchainz
Copy link

I've personally found the terminal window to be a bit too small when there's a lot of terminal text.

Any issues with increasing the size of the window? Or even adding in the option to "maximize"?

This is something I don't mind working on myself and submitting a pull request.

@otavio
Copy link
Member

otavio commented Dec 22, 2020

Hello @markovchainz, absolutely no reason not to consider a PR adding this. We need to consider how we will handle the session recording, but this is a second step.

If you can look at the window resize or maximize option, it'd be great.

@markovchainz
Copy link
Author

Sounds good, I will get started and keep you posted.

@sixhills
Copy link
Contributor

It would also be good if the window could be moved.

@gustavosbarreto
Copy link
Member

@markovchainz

I've personally found the terminal window to be a bit too small when there's a lot of terminal text.

I agree.

Any issues with increasing the size of the window? Or even adding in the option to "maximize"?

The main problem is how to send the 'window resize' to the SSH server through WebSocket since there's no protocol on top of WebSocket connection (currently just raw SSH over WebSocket).

@gustavosbarreto
Copy link
Member

@sixhills

It would also be good if the window could be moved.

This is a limitation of the Vuetify framework, it does not supports movable dialogs (https://vuetifyjs.com/en/components/dialogs/)

@gustavosbarreto
Copy link
Member

gustavosbarreto commented Jan 4, 2021

@markovchainz

Sounds good, I will get started and keep you posted.

If you need it now and can't wait for window resize in Web UI you can use a native SSH client which supports window resizing.

@gustavosbarreto gustavosbarreto added kind/feature New feature or request help-wanted Extra attention is needed labels Jan 4, 2021
@noreng-jg
Copy link
Contributor

noreng-jg commented Jan 4, 2021

The implementation of SSH in Shellhub seems to be in raw bytes, a reference for the JSON version
of the implementation with two types of messages (resize and command)
seems to available in:
https://mojotv.cn/2019/05/27/xtermjs-go#yaurm

@gustavosbarreto
Copy link
Member

The implementation of SSH in Shellhub seems to be in raw bytes, a reference for the JSON version
of the implementation with two types of messages (resize and command)
seems to available in:
https://mojotv.cn/2019/05/27/xtermjs-go#yaurm

This is exactly what needs to be done

@otavio otavio added this to the Future milestone Feb 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help-wanted Extra attention is needed kind/feature New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants