-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 experience #618
Comments
I owe you comments here, and I will provide!
That almost makes it hard to evaluate further, because I don't know what I'm missing! So, I think even just your three items above would go QUITE a long way to improving this. |
One more focused thing you could look at would be the second bullet point. In a dream world where anything is possible, what contextual information would you love to see alongside a search result to help a user know that that’s the result for them? |
Let's see... to throw some wild ideas out there First set of results: the word is a hit for page title/file name Second group of results: the word is in a heading Otherwise, hits for the word in text display:
I don't even know whether categorizing like above is possible, but if so, it might be interesting to also highlight when the word appears in a code sample? Does that give you some ideas to work with? :) |
This is super helpful — thanks Sarah! |
Also adding, it constantly comes up that people click on our language picker instead of the search bar when searching! |
@mayank99 If you are interested in contributing to a less-confusing design, this would be the place to say so! (Just to make sure we get all the search stuff discussed here, and form and function work together.) |
@sarah11918 Yeah, if you have a general idea of what you would like the new design to look like, I can implement it and submit a PR. I was thinking of doing three things:
|
I wonder if the key issue is actually the field width like @mayank99 suggests in the final bullet point — certain expectations between how wide a search box is vs a drop-down. Here’s a simple tweak on that front (would truncate Portuguese but we can do that with I don’t know if we also think the styling is an issue. Flipping the styling these each have currently would look like this: |
I believe using the language's BCP flag is the best approach here. It's short enough and avoids ambiguity. For example, removing "do Brasil" would make people believe it's European Portuguese instead. Thanks for the willingness to tackle this @mayank99, you're awesome! |
I'm not sure about using the BCP tags as I don't know how familiar everyone is with those. For some of our languages that would also mean a label in a different alphabet from that language's main alphabet. I think visually truncating the label for Portuguese but otherwise leaving the content unchanged is ok: "Português d…" |
Agree with this. Languages <=> Flags is not a 1:1 mapping. Plus, with truncation, the user can still see the full name if they open the select menu. |
Just chiming in to say that I like both the change in field with, and the pill/rounded idea for the search bar! |
Hey everyone, I’m one of the people who always clicks the language instead of search. I proposed a solution for that in #1972 but I understand it will be a bad choice for removing the text from the language picker. Now we are left with two options:
Thoughts? |
There’s only one thing outstanding in this issue:
From my research, there’s not much we can do here as we’re constrained by the result data we get back from Algolia. I’m personally feeling pretty good with the position we’re in now, so I’m going to close this as done and we can open new issues for any search questions we run into in the future. |
Have experience with or interested in learning more about DocSearch? Help us out and take a look at one of these tasks!
Increase the number of results returned by DocSearch (currently maxes out at 5). Maybe with the
pagination
parameter passed tosearchParameters
?What’s the number of results we want? Should it paginate to include as many as possible (if that’s possible)?
Improve the snippet for each result so there’s clearer context around what you might be clicking on. (Awaiting more feedback from @sarah11918 on what we’d like there.)
Break up search results into categories: docs vs tutorial
Look into duplicate results. Here’s an example where results show up twice, one always linking to the page’s
#article
ID.22/11/3 — This doesn’t seem to be happening anymore?
Look into items in the integration nav component ranking in search, e.g. Docs "search stats" investigation #2042 (comment)
Update (18 Nov 2022): This should be resolved when the Algolia crawler next runs — I updated the crawler to strip the navigation component from the page before extracting records.
The text was updated successfully, but these errors were encountered: