Skip to content

Conversation

@lihaoyi
Copy link
Member

@lihaoyi lihaoyi commented Apr 16, 2025

If the ANSI escape is split over multiple writes, the current color tracking fails

@lihaoyi lihaoyi force-pushed the main branch 2 times, most recently from 1d3b959 to 3ba698a Compare July 10, 2025 04:00
@lihaoyi
Copy link
Member Author

lihaoyi commented Aug 30, 2025

The issue seems to be that flush() calls writeOutBuffer(), so a single line may be written more than once if broken into multiple writes with flushes in between, and if the ANSI escape code is broken over the multiple writes then the fansi.Str.apply( call does not properly recognize the partial escape codes on each side of the flush

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.

1 participant