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

Update search to DocSearch v3 #3156

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

francoischalifour
Copy link
Contributor

@francoischalifour francoischalifour commented Jul 30, 2020

Hello React team! This PR integrates the new version of DocSearch, that better reflects what we think documentation search should be.

Description

Reducing users' efforts to get them started integrating your product is key to adoption. We created DocSearch in 2015 as a community effort to solve this problem.

We've had time to try different approaches since then and would like to propose you a new documentation search experience.

This new experience has been well received and is now live on the Docusaurus v2 integration: facebook/docusaurus#2815.

Preview

Video preview

Mobile Desktop
Mobile preview Desktop preview

Changes

You can read about all the new features in the Docusaurus PR (facebook/docusaurus#2815), but here's a summary:

  • We replaced the dropdown list for a modal
  • You can open the search modal with Ctrl + K, ⌘ + K or /
  • No hard refresh (we leverage the Gatsby navigation API in this integration)
  • The experience works on mobile and is close to a native mobile experience
  • Recent searches
  • Favorite searches
  • Selection search
  • Search suggestions
  • Offline detection

The bundle size decreased and we now provide React components which can be lazy loaded.

What's next

This new DocSearch version has been used in production for a couple of weeks now, on the DocSearch website and on a few Docusaurus v2 integrations.

This version is marked as alpha for now because the underlying libraries that we use (e.g., autocomplete-core) are still being worked on at @algolia to make sure that it handles more cases than DocSearch and usual autocomplete search experiences. We plan to go stable in the coming weeks but no major changes will happen to DocSearch.

Let us know what you think!

@alexkrolick
Copy link
Collaborator

It seems pretty aggressive to hijack a top-level shortcut (Cmd+K) which may have OS or browser-level bindings. In Firefox that focuses the search box. Anyone else have thoughts on that?

@francoischalifour
Copy link
Contributor Author

francoischalifour commented Aug 3, 2020

@alexkrolick We (the DocSearch team) conducted a study and explained why we decided to go with this keyboard shortcut in facebook/docusaurus#2815 (comment). Let us know if you have further concerns 🙂

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

Successfully merging this pull request may close these issues.

3 participants