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

Pressing enter when searching prevents results from being displayed #1639

Open
ascott18 opened this issue Oct 28, 2022 · 1 comment
Open

Comments

@ascott18
Copy link

Description

If a query is typed out and the Enter key is pressed, the UI reports that the query has no results despite the fact that the AJAX request responded with plenty of results.

Video.mp4

Steps to reproduce

  1. Open the search input
  2. Search for something, typing in the query rapidly and pressing enter when done typing.
  3. See the message that there are no results, despite there actually being results that would have been shown had Enter not been pressed after typing.

Live reproduction:

Any site that uses docsearch. E.g. https://docsearch.algolia.com/. To reproduce you may need to enable throttling of network requests in developer tools if searches are completing before you can manage to press enter.

Expected behavior

Pressing Enter triggers the search immediately (same as waiting a short time after stopping typing) but doesn't otherwise break the displaying of results.

Environment

  • OS: [e.g. Windows / Linux / macOS / iOS / Android] Any
  • Browser: [e.g. Chrome, Safari] Any
  • DocSearch version: [e.g. 3.0.0] 3.3.0
@randombeeper
Copy link
Collaborator

@ascott18 I'm just now going through open issues, sorry for the long long delay.

If network traffic is throttled down, the behavior now seems to be a little different. I've seen that sometimes the model just goes away, other times a page is shown that doesn't even match my search. So there is definitely something that could be done here, while not quite exactly what was described initially, there is still some sketchy behavior. Because of that, I'm going to keep it open.

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

No branches or pull requests

2 participants