gum table v0.14.4 broke height

[uwe-schwarz]

Describe the bug
--height=0 is taken literally and only displays the header row.

Additionally it crashes with pressing two times cursor down.

To Reproduce
Steps to reproduce the behavior:
echo -e "col1,col2,col3\nfoo,bar,baz" | gum table

Expected behavior
Display up to height lines, should default to something like before (10).

Screenshots
If applicable, add screenshots to help explain your problem.
Table:

Crash:

Desktop (please complete the following information):

• OS: macOS 14.6.1
• Terminal: iTerm
• gum version 0.14.4

[GregPK]

Reproduced on the following setup:

Desktop (please complete the following information):

• OS: Archlinux, kernel 6.6.47-1-lts
• Terminal: alacritty
• gum version 0.14.4

[meowgorithm]

Oof, thanks for flagging (and for the great report). We'll get this one patched.

[meowgorithm]

So charmbracelet/bubbles#608 should fix the panic, but it looks like the regression happened in #640 (in response to #554).

It appears that the height is essentially defaulting to 1 now, which of course isn't ideal. What would probably be best is if we defaulted the height to 12 or so, and then used the number of lines in the input if it's less than that.

[hxlnt]

A similar issue is showing up for me on gum file as well; only the first line is shown unless --height is explicitly passed in.

[meowgorithm]

@hxlnt Noted, thank you!