Incorrect intial PTY size when used with Terminal #56
Labels
bug
Something isn't working
Comments
lihop
added a commit
that referenced
this issue
Aug 12, 2022
If a PTY has a terminal_path set to a valid Terminal, then ensure that the initial cols and rows of PTY match the cols and rows of the Terminal when calling fork() or open(), otherwise PTY will output wrong-sized data for the Terminal until resized. Fixes #56.
lihop
added a commit
that referenced
this issue
Aug 14, 2022
If a PTY has a terminal_path set to a valid Terminal, then ensure that the initial cols and rows of PTY match the cols and rows of the Terminal when calling fork() or open(), otherwise PTY will output wrong-sized data for the Terminal until resized. Fixes #56.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When running the terminal example scene the initial PTY size does not match the Terminal size but rather the default of 24x80.
Expected:
Actual:
This results in some undesirable behaviour:
This is fixed after resizing the Terminal (for example, by resizing the application window) so it emits the
size_changed()signal, causing PTY to update to the correct size:The text was updated successfully, but these errors were encountered: