textual icon indicating copy to clipboard operation
textual copied to clipboard

Should widgets scroll to highlighted item when resized?

Open TomJGooding opened this issue 9 months ago • 4 comments

I've been hesitant to create an issue for this, but should widgets automatically scroll to the highlighted item after being resized? Or is this something that is expected to be handled by the developer?

The below example with an OptionList hopefully demonstrates what I mean. Obviously resizing the terminal could also cause the highlighted item to go out of view.

from textual.app import App, ComposeResult
from textual.containers import Horizontal
from textual.widgets import Footer, OptionList, Placeholder


class ExampleApp(App):
    BINDINGS = [
        ("t", "toggle_panel", "Toggle Panel"),
    ]

    CSS = """
    Placeholder {
        width: 2fr;
        display: none;
    }

    .show-panel {
        Placeholder {
            display: block;
        }
    }
    """

    def compose(self) -> ComposeResult:
        with Horizontal():
            yield OptionList(
                *[
                    f"This is option {i} which is long and might wrap when the widget is resized"
                    for i in range(100)
                ]
            )
            yield Placeholder()
        yield Footer()

    def on_mount(self) -> None:
        options = self.query_one(OptionList)
        options.highlighted = 50

    def action_toggle_panel(self) -> None:
        self.toggle_class("show-panel")


if __name__ == "__main__":
    app = ExampleApp()
    app.run()

TomJGooding avatar Feb 25 '25 21:02 TomJGooding

We found the following entries in the FAQ which you may find helpful:

Feel free to close this issue if you found an answer in the FAQ. Otherwise, please give us a little time to review.

This is an automated reply, generated by FAQtory

github-actions[bot] avatar Feb 25 '25 21:02 github-actions[bot]

It probably should... I can't think of any reason why not scrolling would be more desirable. Should be a simple fix. Calling scroll_to_highlight on the resize event.

willmcgugan avatar Feb 26 '25 11:02 willmcgugan

I could see reasons why I wouldn't want this. If I've got the highlighted item scrolled out of view, it might suggest I'm reading other items in the list. What is highlighted doesn't always correspond to where my attention currently is. If I resize in this case, I definitely would want to try and keep my current reading position in the list (in other words: don't scroll).

darrenburns avatar Mar 03 '25 16:03 darrenburns

@darrenburns Fair point. I'm usually navigating with the keyboard, so hadn't considered if you'd scrolled away from the highlight using the mouse.

TomJGooding avatar Mar 03 '25 16:03 TomJGooding