Scrolling in long OptionList doesn't accompany keyboard navigation by default

[rodrigogiraoserrao]

If you use the keyboard to navigate inside an OptionList and if you move past the visible options, there is no scrolling to accompany your movement (there should).

Screen.Recording.2024-01-15.at.11.46.10.mov

App shown in video

from textual.app import App, ComposeResult
from textual.widgets import OptionList
from textual.widgets.option_list import Option


class ListApp(App[None]):
    def compose(self) -> ComposeResult:
        yield OptionList(*[Option(f"This is option #{n}") for n in range(100)])


if __name__ == "__main__":
    ListApp().run()

[davep]

What's the height for this example?

[davep]

Perhaps Why doesn't the DataTable scroll programmatically? needs expanding a wee bit?

[rodrigogiraoserrao]

🤦
Maybe OptionList should come with a max-height: 100% by default?

[TomJGooding]

Maybe OptionList should come with a max-height: 100% by default?

I thought this fix was simple enough that I could try to help, until I remembered that OptionList has padding/border.

This might need a fix for percentage dimensions (#4037 or #4023) to land first.

[github-actions]

Don't forget to star the repository!

Follow @textualizeio for Textual updates.