Skip to content

feat/fix: keyboard shortcuts for search/filter inputs#2476

Open
lidel wants to merge 3 commits intomainfrom
fix/search-filter-escape-clear
Open

feat/fix: keyboard shortcuts for search/filter inputs#2476
lidel wants to merge 3 commits intomainfrom
fix/search-filter-escape-clear

Conversation

@lidel
Copy link
Member

@lidel lidel commented Feb 11, 2026

Small quality of life improvements related to search/filtering on Files and Peers screens.

Changes

  • pressing Escape clears the search filter text on both Files and Peers screens
  • pressing / focuses the filter input (opens it first on Files if hidden)
  • / shortcut is documented in the Keyboard Shortcuts modal (Shift+?)
  • Peers filter input is now a controlled component

Test plan

  • e2e tests pass for Files search filter
  • e2e tests pass for Peers filter
  • manual: on Files, press / to open+focus search, type filter, press Escape to clear, press / again to re-focus
  • manual: on Files, enable search, navigate to Status and back, press / to confirm it focuses
  • manual: on Peers, press / to focus filter, type text, press Escape to clear

add onKeyDown handler to SearchFilter so pressing Escape resets the
filter text and restores the full file listing, and add e2e test
pressing / opens the search filter and focuses the input.
when search is already visible but not focused, / re-focuses it.
the shortcut is documented in the Keyboard Shortcuts modal.

includes e2e tests for open, re-focus, and navigate-away-and-back.
pressing / focuses the Filter peers input, pressing Escape clears it.
the filter input is now a controlled component so Escape can reset
both the input text and the filter state.

includes e2e tests for both behaviors.
@lidel lidel requested a review from a team as a code owner February 11, 2026 01:55
@lidel lidel mentioned this pull request Feb 11, 2026
64 tasks
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