Skip to content
This repository has been archived by the owner on Aug 6, 2023. It is now read-only.

Is there a way to set the amount of rows that is kept above or below the selected item in a list? #328

Closed
ducaale opened this issue Jun 23, 2020 · 2 comments

Comments

@ducaale
Copy link

ducaale commented Jun 23, 2020

This was requested before in #96 (comment)

Additionally, I'd love to have support for a scroll offset. That is, a set amount of rows that is kept above or below the selected row, i.e. the list starts scrolling X rows before you hit the top/bottom. One reason is that this allows you to see whether there are still more rows to come even without a scrollbar.

@ducaale ducaale changed the title Is it there a way to set amount of rows that is kept above or below the selected row in a list? Is there a way to set the amount of rows that is kept above or below the selected item in a list? Jun 23, 2020
@fdehau
Copy link
Owner

fdehau commented Jun 28, 2020

Not at the moment but that should be fairly simple to implement as a customization option on List.

@sayanarijit
Copy link
Contributor

Also tables (ref sayanarijit/xplr#54).

kevinjohna6 added a commit to kevinjohna6/tui-rs that referenced this issue Jul 9, 2022
Consider padding when choosing offset/scrolling to selected item
resolves fdehau#328
kevinjohna6 added a commit to kevinjohna6/tui-rs that referenced this issue Jul 9, 2022
Consider padding when choosing offset/scrolling to selected item
resolves fdehau#328
kevinjohna6 added a commit to kevinjohna6/tui-rs that referenced this issue Jul 9, 2022
Consider padding when choosing offset/scrolling to selected item
resolves fdehau#328
kevinjohna6 added a commit to kevinjohna6/tui-rs that referenced this issue Jul 9, 2022
Consider padding when choosing offset/scrolling to selected item
resolves fdehau#328
kevinjohna6 added a commit to kevinjohna6/tui-rs that referenced this issue Jul 9, 2022
Consider padding when choosing offset/scrolling to selected item
resolves fdehau#328
@fdehau fdehau closed this as completed Aug 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants