jedib0t / go-pretty / 40327139338

100%

main: 100%

Pull #429

github

text: apply the box drawing width override in StringWidth

StringWidth went through runewidth's own StringWidth, which does not know
about the box drawing and block element override that RuneWidth applies, so
the two disagreed in an East Asian locale: StringWidth("┌──┐") came back 8
where RuneWidth says each of those is one cell.

Counted per grapheme cluster through RuneWidth instead, which is runewidth's
own loop plus the override. That also fixes a case the first version of this
patch got wrong: it corrected by subtracting for every box drawing character
in the string, including ones runewidth had never counted, so a Prepend
character joining the box drawing character after it into one cluster lost a
cell -- StringWidth("aツ؀─") came back 3 instead of 4.

With East Asian width off the results are unchanged.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

15 of 15 new or added lines in 1 file covered. (100.0%)

5175 of 5175 relevant lines covered (100.0%)

1.21 hits per line