jedib0t / go-pretty / 43311170021

100%

push

github

table: measure auto-index padding in display columns, not runes (#425)

renderColumnAutoIndex sized the separator row with utf8.RuneCountInString
while every other place that turns Box.PaddingLeft/PaddingRight into a width
budget uses text.StringWidthWithoutEscSequences. The auto-index separator
then disagreed with the auto-index data rows in the same table, in both
directions: padding "　" is 1 rune and 2 columns, so the separator came
out narrow, and padding wrapped in escape sequences is 11 runes and 1 column,
so it came out wide.

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

5191 of 5191 relevant lines covered (100.0%)

1.21 hits per line