feat: filter feature works

[alswl]

Request #5 is done.

demo go run examples/filter/main.go.

A filtered simple default table
Currently filter by:
Press q or ctrl+c to quit

┏━━━━━━━━━━━━━┳━━━━━━━━━━━━━┳━━━━━┓
┃        Title┃       Author┃Desc…┃
┣━━━━━━━━━━━━━╋━━━━━━━━━━━━━╋━━━━━┫
┃Computer Sys…┃Randal E. Br…┃This…┃
┃Effective Ja…┃ Joshua Bloch┃The …┃
┃Structure an…┃Harold Abels…┃Stru…┃
┃Game Program…┃Robert Nystr…┃The …┃
┣━━━━━━━━━━━━━┻━━━━━━━━━━━━━┻━━━━━┫
┃                              1/1┃
┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛

A filtered simple default table
Currently filter by:
Press q or ctrl+c to quit

┏━━━━━━━━━━━━━┳━━━━━━━━━━━━━┳━━━━━┓
┃        Title┃       Author┃Desc…┃
┣━━━━━━━━━━━━━╋━━━━━━━━━━━━━╋━━━━━┫
┃Computer Sys…┃Randal E. Br…┃This…┃
┃Structure an…┃Harold Abels…┃Stru…┃
┃Game Program…┃Robert Nystr…┃The …┃
┣━━━━━━━━━━━━━┻━━━━━━━━━━━━━┻━━━━━┫
┃                        /pro  1/1┃
┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛

┏━━━━━━━━━━━━━┳━━━━━━━━━━━━━┳━━━━━┓
┃        Title┃       Author┃Desc…┃
┣━━━━━━━━━━━━━╋━━━━━━━━━━━━━╋━━━━━┫
┃Game Program…┃Robert Nystr…┃The …┃
┣━━━━━━━━━━━━━┻━━━━━━━━━━━━━┻━━━━━┫
┃                       /game  1/1┃
┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛

[Evertras]

Thanks for the submission! A few things to look at.

I also want to consider how a user might supply filters/searches through the API. For example, a user might have a text box outside the table to supply a filter term. Can we add a way to supply the filter term as well as the text input you've added? I do really like the / as I live in vim and that feels natural to me. :D

There will be room for expanding this in the future for things like numerical comparisons/filters, but I think that can definitely be another PR as long as we leave room for it.

[Evertras]

Just a note, we also need to make sure to fix the linting/testing coverage. I unfortunately do want to stay strict with that, as it's caught many of my own mistakes even in this project.

[alswl]

Just a note, we also need to make sure to fix the linting/testing coverage. I unfortunately do want to stay strict with that, as it's caught many of my own mistakes even in this project.

Sure.

[Evertras]

Hey there, just checking if you were able to add some coverage. If not, I'm willing to help out here. There appear to be some existing bugs regarding the selectable rows that snuck in unrelated to this PR, and another issue that would collide with the keymap added in this PR, so I'd like to unblock this if you find the test coverage requirements painful.

[alswl]

@Evertras I added as more unit tests as I can. And I merged with upstream main branch(solved some conflicts).

If the coverage is still under the required, help is wanted~

[Evertras]

Thanks! Just a few linting things snuck in... a quick make lint locally should point them out as well. Once those are fixed I'm happy to merge!

[alswl]

@Evertras thanks for point out. Style issues were fixed.

[Evertras]

Thank you for the patience, I'll take care of the coverage gap; I'll just not make a release until I fix that. Appreciate the work!

[alswl]

@Evertras Maybe availableRows is better than visibleRows.