-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
iTerm2 clear screen does not clear the screen properly #5977
Comments
This issue also came up during #1183 (comment) and it also happens on Jest #3661 (comment). |
Hello @kxalex. Please provide a minimal reproduction using a GitHub repository or StackBlitz (you can also use examples). Issues marked with |
I'm unable to reproduce this issue anymore at all. Please include a minimal test case that could be used to reproduce this issue. |
iTerm 3.5.2 cloudflare supports 1.5.3 for now, but you can change to 1.6.0 and will get the same result. Also adding demo video where the half of the screen is not-cleared because those values needs to be flipped. Screen.Recording.2024-06-29.at.7.28.55.PM.mp4 |
Thanks for the setup, I can reproduce the issue now. Also worth to note that this issue seems to only come up in iTerm2. Mac's default Terminal and VSCode's terminal seem to work just fine. |
Describe the bug
iTerm2 does not clear the screen properly when vitest is running in watch mode.
Reproduction
Use iTerm2 under Mac.
Run vitest on any project and on re-run not whole screen was cleared, there are some leftovers from previous run.
vitest currently uses to clear the screen and erase scrollback.
iTerm2 reacts properly on that sequence, although clear screen sequence leaves 1 page visible and erase scrollback removes the history leaving this not-cleared page.
By flipping the sequence iTerm2 behaves properly.
I've added changes in this PR against 1.6.0 version. Although I'm still using 1.5.3 due to cloudflare limitations.
https://github.com/vitest-dev/vitest/pull/5978/files
The erase scrollback and clear screen can be tested in shell using
This was also tested on default macos terminal and it works fine too.
System Info
Used Package Manager
pnpm
Validations
The text was updated successfully, but these errors were encountered: