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

Display glitched when switching sessions with session manager on windows with WSL #2736

Closed
tranzystorekk opened this issue Aug 28, 2023 · 6 comments · Fixed by #2750
Closed

Comments

@tranzystorekk
Copy link
Contributor

When creating a new session or switching to another existing one with the session manager, the display ends up in a broken state:

obraz

In Case of Graphical or Performance Issues

  1. Delete the contents of /tmp/zellij-1000/zellij-log, ie with cd /tmp/zellij-1000/ and rm -fr zellij-log/ (/tmp/ is $TMPDIR/ on OSX)
  2. Run zellij --debug
  3. Run stty size, copy the result and attach it in the bug report
  4. Recreate your issue.
  5. Quit Zellij immediately with ctrl-q (your bug should ideally still be visible on screen)

Please attach the files that were created in /tmp/zellij-1000/zellij-log/ to the extent you are comfortable with.

Basic information

zellij --version: 0.38.0
stty size: 39 171
uname -av or ver(Windows): Linux DESKTOP-0QOMDS0 5.15.90.1-microsoft-standard-WSL2 #1 SMP Fri Jan 27 02:56:13 UTC 2023 x86_64 GNU/Linux

List of programs you interact with as, PROGRAM --version:
Windows Terminal Version: 1.17.11461.0

Further information
Reproduction steps, noticeable behavior, related issues, etc

zellij.log

@imsnif imsnif changed the title Display glitched when switching sessions with session manager Display glitched when switching sessions with session manager on windows with WSL Aug 29, 2023
@imsnif
Copy link
Member

imsnif commented Aug 29, 2023

@tranzystorek-io - could I rely on you for some help troubleshooting? I do not have access to a windows machine and cannot recreate this issue myself.

Right now I'm suspecting one of two things:

  1. Windows terminal being creative about an alternative screen switch (CSI ? 1049h) when already in an alternative screen
  2. Somehow misreported window size for the terminal window

I created a debug branch to this repo called glitch-debug (https://github.com/zellij-org/zellij/tree/glitch-debug). Would you be willing to run it and see if you can recreate the issue? Also please send me the logs it creates (normal zellij.log file) as I put some messages there to help troubleshoot.

Thanks!

@tranzystorekk
Copy link
Contributor Author

THe issue does not reproduce on the glitch-debug branch, something in that specific commit (2929dec) prevents repro. Checked that the issue still reproduces on the previous commit (24ce348)

@imsnif
Copy link
Member

imsnif commented Aug 30, 2023

Thanks for testing it out @tranzystorek-io ! I tidied up that branch and added some stuff. Could you do me a favor and give it one more test to make sure it's still working?

Basically, I'm pretty sure the issue here is Windows Terminal not behaving properly when receiving a CSI ? 1049h (as I suspected in point 1 above). This issue works around it by not sending it on a reconnect (we don't need to anyway, it was just there to save some code).

@imsnif
Copy link
Member

imsnif commented Aug 30, 2023

Once you confirm I'll merge the branch and release a patch version (along with some more minor fixes).

@tranzystorekk
Copy link
Contributor Author

I can confirm the new commit on the glitch-debug branch still works (does not reproduce) 😊

@imsnif
Copy link
Member

imsnif commented Aug 30, 2023

Thanks for the super quick confirmation!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants