charmbracelet / lipgloss / 22954902143

65%

push

github

fix(table): height and overflow with wrapping content (#620)

The current algorithm which calculates the first and last visible rows
does not take into account content wrapping (and has a number of other
problematic edge cases).

This is partially due to `expandRowHeights` actually not updating
`rowHeights`, but also due to a simplified implementation which does not
try to take them into account.

Rewrite the visible rows calculation to take content wrapping into
account, making sure we consider the actual row height that will be
used when rendering the table.

59 of 61 new or added lines in 3 files covered. (96.72%)

2630 of 4063 relevant lines covered (64.73%)

459.75 hits per line