New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: make the paginator key bindings customizable #272
Conversation
paginator/paginator.go
Outdated
// UsePgUpPgDownKeys is deprecated. Customize KeyMap instead. | ||
UsePgUpPgDownKeys bool | ||
UseLeftRightKeys bool | ||
UseUpDownKeys bool | ||
UseHLKeys bool | ||
UseJKKeys bool | ||
// UseLeftRightKeys is deprecated. Customize KeyMap instead. | ||
UseLeftRightKeys bool | ||
// UseUpDownKeys is deprecated. Customize KeyMap instead. | ||
UseUpDownKeys bool | ||
// UseHLKeys is deprecated. Customize KeyMap instead. | ||
UseHLKeys bool | ||
// UseJKKeys is deprecated. Customize KeyMap instead. | ||
UseJKKeys bool |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From https://go.dev/blog/godoc
Sometimes a struct field, function, type, or even a whole package becomes redundant or unnecessary, but must be kept for compatibility with existing programs. To signal that an identifier should not be used, add a paragraph to its doc comment that begins with “Deprecated:” followed by some information about the deprecation.
This looks good to me behaviour wise since I did a quick code search on GitHub repos using the |
0d7e4e2
to
1315fd7
Compare
1315fd7
to
dd720c6
Compare
Awesome work @knz, thank you! |
Fixes #271.
This will also make the paginator benefit from the improvement in #261.
cc @maaslalani @meowgorithm