Skip to content

support search in select component dropdown#70

Merged
lanzhenw merged 4 commits intodevelopfrom
improve_select_search
Feb 10, 2026
Merged

support search in select component dropdown#70
lanzhenw merged 4 commits intodevelopfrom
improve_select_search

Conversation

@lanzhenw
Copy link
Contributor

Rationale

Select component should allow arrow selection + search among the option dropdown

Changes

Screen.Recording.2026-02-10.at.2.23.43.PM.mov

Testing

@lanzhenw lanzhenw requested a review from erik-nieh February 10, 2026 20:27
@lanzhenw lanzhenw self-assigned this Feb 10, 2026
@lanzhenw lanzhenw requested a review from a team as a code owner February 10, 2026 20:27
() =>
query
? options?.filter((opt) =>
opt.data.label.toLowerCase().includes(query.toLowerCase())
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe trim() this query too

onClose={() => setQuery("")}
>
<ComboboxInput
displayValue={getDisplayValue}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we be displaying the query to the user?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should show what's currently selected rather than the query itself

@lanzhenw lanzhenw requested a review from erik-nieh February 10, 2026 21:38
@lanzhenw lanzhenw merged commit 5c58be2 into develop Feb 10, 2026
1 check passed
@lanzhenw lanzhenw deleted the improve_select_search branch February 10, 2026 22:49
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