Skip to content

feat(ui): add search filter to provider grid - #10437

Merged
The-Best-Codes merged 3 commits into
aaif-goose:mainfrom
harrykamboj1:feat/provider-grid-search
Jul 14, 2026
Merged

feat(ui): add search filter to provider grid#10437
The-Best-Codes merged 3 commits into
aaif-goose:mainfrom
harrykamboj1:feat/provider-grid-search

Conversation

@harrykamboj1

Copy link
Copy Markdown
Contributor

Summary

Adds a search input to the Providers grid so users can quickly filter
providers by name or description instead of scanning the full card grid.

  • New search input with a Search icon above the grid
    (data-testid="provider-search-input")
  • Case-insensitive filtering on provider display_name and description
    (query is trimmed)
  • "No providers match "{query}"" empty-state message when a search yields
    no results
  • New i18n messages: providerGrid.searchPlaceholder, providerGrid.noMatch
  • Moved the GridLayout wrapper from ProviderGrid into ProviderCards
    so the search bar and empty-state text render outside the grid

Applies to both the onboarding provider selection flow and the settings
page, since both render ProviderGrid.

Testing

  • cd ui/desktop && pnpm run typecheck — passes
  • cd ui/desktop && pnpm test — passes
  • Manual testing via just run-ui → Settings → Providers:
    • Empty query renders the full sorted grid, unchanged from today
    • Typing filters cards live by name and description (case-insensitive)
    • A query with no matches shows the empty-state message
    • Clearing the query restores the full grid
    • Verified in both light and dark mode

Related Issues

Relates to #10436
Discussion: N/A

Screenshots/Demos (for UX changes)

Screenshot 2026-07-14 at 7 42 32 AM

@The-Best-Codes The-Best-Codes self-assigned this Jul 14, 2026
@The-Best-Codes
The-Best-Codes added this pull request to the merge queue Jul 14, 2026
Merged via the queue into aaif-goose:main with commit c1eebf0 Jul 14, 2026
22 checks passed
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.

2 participants