-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Improve search response #302
Comments
@erquhart Can you explain why this was closed? I still think it is unclear that you have to push enter to search. |
Busted. Maybe we just shouldn't have an enter button at all, just a 1 second debounce (in case an integration is in use). |
I would be fine with an instant search like you mentioned, because of the small usable space we have, although I still think that is less clear. |
Ideally I think it would look like this for all backends:
|
What's React Toolbox? |
That's an old comment, we used to use a library called React Toolbox but it was removed in 1.0. |
Many interfaces don't, it's a fair point. |
I think we should have one of two things here:
Right now, if I go to type in the search box and don't think to press "Enter", I get nothing: no feedback at all on what do do. |
I agree with @tech4him1, we should go for As we are discussing search, how about we add Algolia, where users can add API keys and search bar will be replaced with Algolia search bar. Where users can not only search the title, but also the content inside each post. |
Instant search is a more intuitive approach. Submission is implicit, which is why I created this issue in the first place, but it's a common enough pattern that it shouldn't be a problem. There's a custom Algolia integration already in the CMS, it just isn't a public feature yet. That's pending backend/integration API redesign work that's currently underway. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Requirements
The text was updated successfully, but these errors were encountered: