Skip to content
This repository has been archived by the owner on Dec 30, 2022. It is now read-only.

fix(searchbox): prevent concurrent query updates from state while input is focused #1133

Merged
merged 2 commits into from
Jul 19, 2022

Conversation

dhayab
Copy link
Member

@dhayab dhayab commented Jul 18, 2022

JIRA: FX-1552

Fixes #1123

This PR prevents state from overriding the query value in a search box if it still has focus. This can happen in a search-as-you-type environment when network speed is low and a user keeps on typing a query while requests are resolved.

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jul 18, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 6aa78c6:

Sandbox Source
vue-instantsearch-e-commerce Configuration

@dhayab dhayab requested review from a team, sarahdayan and francoischalifour and removed request for a team July 18, 2022 15:58
Copy link
Contributor

@Haroenv Haroenv left a comment

Choose a reason for hiding this comment

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

as far as I can tell the failure mode with slow network doesn't happen in this PR

@dhayab dhayab merged commit c468c0a into master Jul 19, 2022
@dhayab dhayab deleted the fix/searchbox-concurrent-query-updates branch July 19, 2022 12:47
Haroenv pushed a commit to algolia/instantsearch that referenced this pull request Dec 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Vue - Lag Typing (Meili Search)
2 participants