-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Certain escape sequences in quick succession break rendering. #2126
Comments
@agronick Can repro it, seems RIS wrongly recreates the write buffers as well forgetting stuff behind. |
Thanks for taking care of that so quick. You guys are awesome. Any idea when it will hit NPM? |
Prolly with the next release around first week in June. Care to test if #2127 solves the issue for you? Because that code was not touched for 2 years, you either were on a really old version or its not the right fix yet. (at least works with the data you gave in the issue description) |
I just got home for the long weekend and I won't have access until Tuesday. I'll be sure to test it then. It is possible we are on an old version. It is also possible we have the issue in production but most servers don't output fast enough to cause the issue. Or that the users are seeing it but never reported it. Maybe they open and close the terminal until they get lucky and it works. I've never heard anyone mention it. |
@jerch Good news! It definitely fixes it. |
Details
Steps to reproduce
If you take the following escape sequences and run them in a fast loop nothing is rendered in the end.
But if you make them run as a microtask it renders fine:
The same script that contains these escape codes is run daily with thousands of users and it displays fine using an older version of xterm.js.
The text was updated successfully, but these errors were encountered: