Hitting Enter uses the default search page

[humitos]

Steps to reproduce it

1. go to https://docs.readthedocs.io/en/latest/
2. click on search
3. type configuration
4. hit Enter

Actual result

You are redirected to https://docs.readthedocs.io/en/latest/search.html?q=configuration&check_keywords=yes&area=default which is the default search (instead of the Search As You Type)

Expected result

I was expecting that it will just send the current query to the backend and do nothing, basically.

In case we want to have a fallback to the default search result, I think the usage of Enter has to be communicated in the UI.

[dojutsu-user]

This is the default behavior as of now.
As soon as the user starts typing... He is updated of the current status, in this case -- a message is shown to him -- Searching .....

I think it will be okay to mention this in docs?

[stsewd]

I think we should select the first result, so when pressing enter the user can go to the page of that result.

[dojutsu-user]

@stsewd
I am not sure that I followed your comment.
Can you explain a little more.

[stsewd]

When searching, the popup window should have the first result selected, so when hitting enter, the user goes to the first result. No need to use arrow keys, tab or the mouse to select the first result.

edit: looks like is only possible to navigate the results with tab

[dojutsu-user]

@stsewd
I don't think setting the first result as default will be good UI.

Also, it is possible to select the results with the ArrowUp and ArrowDown keys.

[stsewd]

Also, it is possible to select the results with the ArrowUp and ArrowDown keys.

Sorry, I had an extension overriding that.

[dojutsu-user]

I would like to know what will be good UI.
@readthedocs/core ?

[agjohnson]

Normal search function is to perform a search when hitting enter in a search box, we should preserve this. I expect that the search modal results will be updated when I hit enter. I don't expect to be dropped back into the default sphinx search results.

[dojutsu-user]

@agjohnson
Our goal was to provide search-as-you-type capability -- I believe that user shouldn't have to hit enter to get the results.

[stsewd]

I'm fixing this in #81. The idea, is to select the first result, but have a link to the original search, this also helps in the case the api is down, so users can still go to the default search.

[agjohnson]

Our goal was to provide search-as-you-type capability -- I believe that user shouldn't have to hit enter to get the results.

Sure, but they are, myself included 🙂

If the goal is to block the user from hitting enter, then we should capture this keypress and stop the event from propagating. It is unexpected UX to hit enter, whether accidentally or because you're used to text boxes accepting enter keypress as an action, and being dropped into a different UI.

I'm fixing this in #81. The idea, is to select the first result, but have a link to the original search, this also helps in the case the api is down, so users can still go to the default search.

@stsewd I think blocking the event from propagating/default event is fine. We just don't want to drop the user on the search.html page, I would be surprised to select the first search result automatically.

[stsewd]

I would be surprised to select the first search result automatically.

But having enter do nothing feels weird, having the first item selected by default, so you can use enter to go the page is used by several applications (slack, algolia).