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

Searchbar for Arrow Docs #898

Merged
merged 5 commits into from Jun 20, 2018
Merged

Searchbar for Arrow Docs #898

merged 5 commits into from Jun 20, 2018

Conversation

calvellido
Copy link
Member

This PR adds search functionality to Arrow Docs.

It is based on the Algolia DocSearch service, which is offered for free for open source projects. But in case we want to, we can run the scraper ourselves.

General documentation is here:

https://community.algolia.com/docsearch/documentation/docsearch/introduction/

The scraper configuration for Arrow Docs can be found here:

https://github.com/algolia/docsearch-configs/blob/master/configs/arrow.json

A PR can be submitted there in case we want to change the way the docs are analyzed, following the guidelines detailed here. The underlying crawler runs every 24h.

The SASS has been added to our build process, and the JavaScript has been compiled from Algolia sources and minified. The repositories containing the source code for this can be found on:

https://github.com/algolia/docsearch
https://github.com/algolia/autocomplete.js

Completed tasks:

  • Adds a search input into the docs
  • Adds Arrow visual identity adapted styling for the dropdown results
  • Compile and include Docsearch JavaScript code associated with that input
  • Hide unnecessary X on search input when rendered on Chromium or Edge browsers
  • Include autofocus feature when an Arrow Doc page is loaded

captura de pantalla de 2018-06-20 12-37-59

This closes #808

Copy link
Member

@JorgeCastilloPrz JorgeCastilloPrz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, incredible job, it looks amazing! This is gonna become really handy for people browsing the docs. Great job @calvellido! 👏 👏 👏 👏

@calvellido calvellido merged commit e439bb3 into master Jun 20, 2018
@calvellido calvellido deleted the jv-808-searchbar branch June 20, 2018 11:26
RawToast pushed a commit to RawToast/kategory that referenced this pull request Jul 18, 2018
* Add proper styling for the searchbar

* Add Arrow styling for the search results dropdown

* Hide unnecessary X on search input elements on Chromium and Edge

* Add JS code to call Algolia docsearch API. Including autofocus feature.
@calvellido calvellido added docs and removed docs labels Oct 17, 2018
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.

Searchbar for docs
2 participants