Skip to content

Fix search input/query search coupling#385

Open
karlshea wants to merge 1 commit intotimc1:mainfrom
karlshea:search-query
Open

Fix search input/query search coupling#385
karlshea wants to merge 1 commit intotimc1:mainfrom
karlshea:search-query

Conversation

@karlshea
Copy link

@karlshea karlshea commented Nov 26, 2025

Fixes #384

It doesn't seem like the local state for the input is necessary. This changes it to be controlled directly by the current search query, and then the onChange updates that query.

  • Fixes query.setSearch(‘value’) not working
  • Fixes typing to find an action, hitting enter to select it, and the clearing the input to find the next child action

- Fixes query.setSearch(‘value’) not working
- Fixes typing to find an action, hitting enter to select it, and the clearing the input to find the next child action
@vercel
Copy link

vercel bot commented Nov 26, 2025

@karlshea is attempting to deploy a commit to the Tim's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

KBarSearch's input value can't be reliably controlled

1 participant