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

Only auto-scroll terminal on new output when at the bottom #204

Merged
merged 4 commits into from
Dec 11, 2016
Merged

Only auto-scroll terminal on new output when at the bottom #204

merged 4 commits into from
Dec 11, 2016

Conversation

schrej
Copy link
Member

@schrej schrej commented Dec 9, 2016

This updates jquery.terminal to version 0.11.23 to enable new (requested) features.

The server terminal now only automatically scrolls when it is scrolled all the way down.

If there is new output that is not visible a small notification appears at the bottom right. A click on it will scroll all the way down.

fixes #153

@schrej schrej changed the title Only scroll terminal at the bottom on new output Only scroll terminal to the bottom on new output Dec 9, 2016
@schrej schrej changed the title Only scroll terminal to the bottom on new output Only auto-scroll terminal on new output when at the bottom Dec 9, 2016
@schrej
Copy link
Member Author

schrej commented Dec 10, 2016

There are some issues with Chrome currently, I would put this on hold (but keep the PR open) until this is resolved, then i will update this.

set new scrollBottomOffset to 5 to hopefully resolve issues with chrome
@BentHaase
Copy link

BentHaase commented Dec 10, 2016

With the latest addition (d844c41) all Chrome issues on Ubuntu have been resolved. Works flawless.

@schrej
Copy link
Member Author

schrej commented Dec 11, 2016

Perfect. In my opinion it's ready to merge then.

@DaneEveritt DaneEveritt merged commit c3abb32 into pterodactyl:develop Dec 11, 2016
@schrej schrej deleted the feature/terminal-only-scroll-at-bottom branch December 11, 2016 00:10
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.

Server console should only auto-scroll at the bottom
3 participants