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

refactor: update to ratatui v0.28 #172

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

refactor: update to ratatui v0.28 #172

wants to merge 2 commits into from

Conversation

EdJoPaTo
Copy link
Owner

@EdJoPaTo EdJoPaTo commented Jul 9, 2024

A fix in unicode-width 0.1.13 introduces buggy behavior in ratatui rendering. MQTT topics which publish with control sequences (like espHome devices on /debug do) cause the rendering to be off. Block borders are rendered on wrong positions and colors leak into wrong positions including the next line. (See ratatui/ratatui#1211)

This should be fixed before updating to unicode-width >=0.1.13 (and therefore ratatui > 0.26).

@EdJoPaTo
Copy link
Owner Author

EdJoPaTo commented Jul 30, 2024

Wait for a release which includes ratatui/ratatui#1226

Until then af67ef0 prevents these glitches from happening.

@EdJoPaTo EdJoPaTo changed the title refactor: update to ratatui v0.27 refactor: update to ratatui v0.28 Aug 7, 2024
@EdJoPaTo EdJoPaTo closed this Aug 7, 2024
@EdJoPaTo EdJoPaTo deleted the ratatui-v0.27 branch August 7, 2024 14:57
@EdJoPaTo EdJoPaTo restored the ratatui-v0.27 branch August 7, 2024 14:58
@EdJoPaTo EdJoPaTo reopened this Aug 7, 2024
@EdJoPaTo
Copy link
Owner Author

EdJoPaTo commented Aug 7, 2024

GitHub GUI branch rename pushes it and deletes the old one, killing the PR… hoped for something more useful… 😒

@EdJoPaTo
Copy link
Owner Author

EdJoPaTo commented Aug 7, 2024

Wait for a ratatui release which includes ratatui/ratatui#1312 🤡

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