Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow Persistent Results #705

Merged
merged 2 commits into from
Mar 23, 2021
Merged

Allow Persistent Results #705

merged 2 commits into from
Mar 23, 2021

Conversation

haydncomley
Copy link
Contributor

Results from the search can be made to persist after the blur event has been fired and focus from the input is lost.
This is helpful in a variety of situations where the on screen keyboard needs to be hidden or focus pulled elsewhere with the results from the search still visible.

- Results from the search can be made to persist after the blur event has been fired and focus from the input is lost.
Copy link
Collaborator

@bell-steven bell-steven left a comment

Choose a reason for hiding this comment

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

Can you add keepResultsAfterBlur: false to the default props.

@bell-steven bell-steven merged commit 2e3edf5 into FaridSafi:master Mar 23, 2021
@bell-steven
Copy link
Collaborator

Thanks for the PR!

@haydncomley
Copy link
Contributor Author

No problem at all - i'm just glad my first PR experience was a nice one so thank you too!

@ezemgaray
Copy link

Useful and I think it reduces the autocomplete billing per request !! waiting for release!! @bell-steven @haydncomley

@haydncomley
Copy link
Contributor Author

@ezemgaray

This was completed and merged into master - should be available to use right now as long as you've pulled the last version I believe.

@bell-steven
Copy link
Collaborator

It hasn't been release on npm yet, I am waiting for another commit before releasing.

@bell-steven
Copy link
Collaborator

Sorry for the delay in getting this out.
Released as v2.3.0

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.

3 participants