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

Add search hints/autocomplete #10554

Closed
tofumatt opened this issue Jul 11, 2017 · 9 comments · Fixed by mozilla/addons-frontend#2971
Closed

Add search hints/autocomplete #10554

tofumatt opened this issue Jul 11, 2017 · 9 comments · Fixed by mozilla/addons-frontend#2971

Comments

@tofumatt
Copy link
Contributor

Add a search hint/autocomplete as per mocks. Relies on #3521.

@willdurand
Copy link
Member

The API endpoint has been implemented in mozilla/addons-server#5930, and the API doc is here: https://addons-server.readthedocs.io/en/latest/topics/api/addons.html#autocomplete.

@willdurand willdurand self-assigned this Aug 16, 2017
@willdurand
Copy link
Member

willdurand commented Aug 16, 2017

I have started the UI based on the Sketch files. Things have slightly changed but I tried to stick to the original:

Open to view image

capture d ecran - 2017-08-16 a 12 55 13

@willdurand
Copy link
Member

willdurand commented Aug 18, 2017

@pwalm Hi, I am a bit confused by the new mockups. I have exported the latest version below:

search

What is the first set of entries/suggestions? In the previous mockups, it was about suggestions to help users type right, but now it seems to be different. At first glance, I thought this first set represented a history (previous/past searchs).

Yet, because there is also a second set with icons and arrows, I am not sure about what the first set actually is. If there is an arrow and the icon of a well identified add-on, then I would expect to redirect to the add-on page directly on click/tap, and not to fill the input and trigger the search process.

Could you please give more details about the autocomplete/auto-suggest feature? Thank you!

@kumar303
Copy link
Contributor

I think the first set of menu items are suggestions for search terms and the second set are suggestions for search results.

However, the API only returns suggested search results so let's start by implementing that.

@willdurand
Copy link
Member

Updating milestone.

@pwalm
Copy link
Contributor

pwalm commented Aug 24, 2017

@kumar303 Is correct! First three: search terms. Second three: search results. I can actually make this clearer by adding subheads if we need to, we've done it for some other drop-downs in the header already:
screen shot 2017-08-24 at 4 00 01 pm

@tofumatt
Copy link
Contributor Author

@pwalm Yeah, I think the subheaders make it a lot clearer, and since we use them elsewhere in similar elements I say 👍

@kumar303
Copy link
Contributor

@diox if you take a look at the mock in #10554 you'll see that the first three items are suggested search terms (not results). I assume that would be pretty tricky to implement in the API since we don't currently keep a record of what people typed (other than in request logs).

@pwalm could you add a little bit of rationale for including suggested terms? Would these potentially help users find what they're looking for in ways that suggested results would not?

@ValentinaPC
Copy link

Verified the fix from #3233 using FF55(Win 7, Android 7.2.1)
Search suggestions are displayed.
Postfix videos:
videotogif_2017 08 29_11 15 24
search-desktop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants