docs(table): note that `Offset` must be set after `Rows` or it does nothing. #371

bashbunni · 2024-09-20T19:52:27Z

If the table's rows are not set before Offset, it does nothing. This makes sense, but should be noted in the GoDoc

This works:

table := New().
	Rows(rows...).
	Offset(80).
	Height(45)

This doesn't

table := New().
	Offset(80).
	Rows(rows...).
	Height(45)

bashbunni · 2024-10-21T21:23:18Z

Not sure if this is a thing for lists too 🤔 I've only seen this with tables, haven't tried to break lists...

bashbunni added the documentation Improvements or additions to documentation label Sep 20, 2024

bashbunni changed the title ~~docs(table): note that Offset() must be set *after* Rows or it does nothing.~~ docs(table): note that Offset must be set *after* Rows or it does nothing. Sep 20, 2024

bashbunni linked a pull request Oct 21, 2024 that will close this issue

docs: include warning about table offset #408

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

docs(table): note that `Offset` must be set after `Rows` or it does nothing. #371

docs(table): note that `Offset` must be set after `Rows` or it does nothing. #371

bashbunni commented Sep 20, 2024

bashbunni commented Oct 21, 2024

docs(table): note that Offset must be set *after* Rows or it does nothing. #371

docs(table): note that Offset must be set *after* Rows or it does nothing. #371

Comments

bashbunni commented Sep 20, 2024

bashbunni commented Oct 21, 2024

docs(table): note that `Offset` must be set after `Rows` or it does nothing. #371

docs(table): note that `Offset` must be set after `Rows` or it does nothing. #371