Skip to content

Fix wide_msg truncation with a colored message#740

Merged
djc merged 1 commit intoconsole-rs:mainfrom
glehmann:gln/widebar-truncation-fix-pvlv
Oct 26, 2025
Merged

Fix wide_msg truncation with a colored message#740
djc merged 1 commit intoconsole-rs:mainfrom
glehmann:gln/widebar-truncation-fix-pvlv

Conversation

@glehmann
Copy link
Contributor

ANSI control sequences were not taken into account.

@glehmann glehmann force-pushed the gln/widebar-truncation-fix-pvlv branch 3 times, most recently from 2a046cb to 747671c Compare October 23, 2025 15:11
Copy link
Member

@djc djc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Much better!

@glehmann glehmann marked this pull request as draft October 23, 2025 16:03
@glehmann
Copy link
Contributor Author

I think there would be a bug with unicode combining diacritical marks. I'm checking

@glehmann glehmann force-pushed the gln/widebar-truncation-fix-pvlv branch from 747671c to b0406af Compare October 23, 2025 16:15
@glehmann
Copy link
Contributor Author

I think there would be a bug with unicode combining diacritical marks. I'm checking

Fixed and tested :)

@glehmann glehmann marked this pull request as ready for review October 23, 2025 16:15
@glehmann glehmann changed the title Fix wide_bar truncation with a colored message Fix wide_msg truncation with a colored message Oct 23, 2025
ANSI control sequences were not taken into account.
@glehmann glehmann force-pushed the gln/widebar-truncation-fix-pvlv branch from b0406af to 50e2651 Compare October 23, 2025 17:20
@djc djc merged commit 01957a7 into console-rs:main Oct 26, 2025
10 checks passed
@djc
Copy link
Member

djc commented Oct 26, 2025

Thanks!

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.

2 participants