Skip to content

Navigate filtered nodes with arrow keys + smaller cleanup#208

Open
eibx wants to merge 1 commit into
Maxteabag:mainfrom
eibx:feature/filter-navigation
Open

Navigate filtered nodes with arrow keys + smaller cleanup#208
eibx wants to merge 1 commit into
Maxteabag:mainfrom
eibx:feature/filter-navigation

Conversation

@eibx
Copy link
Copy Markdown

@eibx eibx commented May 20, 2026

I've added the option to use up and down arrows to select the previous and next filter matches.

I've also removed code relating to _tree_filter_typing.

As far as I can see, _tree_filter_typing and _tree_filter_visible are always in sync (always both true or false).

And because of this an early exit in on_key:

if not self._tree_filter_visible:
    # Pass to next mixin in chain (e.g., AutocompleteMixin)
    super().on_key(event)  # type: ignore[misc]
    return

I can't see how any of the not self.tree_filter_typing statement could be reached.

Again thanks for your work 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant