-
Notifications
You must be signed in to change notification settings - Fork 1k
Fix: Increase the number of search results #2716
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah good find! I didn't even realize we were hitting up on the limit. By changing this does it mean that no matter what 7 items will always be shown, or only when there are more that 5 matching?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok I'm cool with this, but let's get a couple more opinions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I also agree with this change. It would be good to also have a way to see the complete list of results. Is that possible?
So maybe I'm wrong but I tested locally and nothing changes when I change this value. Is it the fact that it is local, or do I have to use the algolia API key. I don't know but I'll look into it. |
Let's optimistically merge and see what happens. |
So, I have just made some corrections and research. Is it possible to reopen this pull request or should I create a new one? |
PRs cannot be reopened. Followup PRs are in this area are welcome. |
As the Scala documentation grows (the new Scala3 version, earlier versions, etc.) the search areas become more diverse, and a search should be as comprehensive as possible so that even the most novice user can easily find what they are looking for.
This is why in this PR, I have increased the number of search results.
Fixes #2715