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 Algolia search #76

Merged
merged 2 commits into from
May 1, 2018
Merged

add Algolia search #76

merged 2 commits into from
May 1, 2018

Conversation

orbeckst
Copy link
Member

@orbeckst orbeckst commented May 1, 2018

- closes #73
- uses Algolia free plan with configuration at
  https://github.com/algolia/docsearch-configs/blob/master/configs/mdanalysis.json
- Algolia index is updated every 24 hours.
- NOTE: needed to set site.baseurl to empty to make local testing work;
  according to https://byparker.com/blog/2014/clearing-up-confusion-around-baseurl/
  we do NOT need to use it.

  When running locally my installation failed to load local js and only after
  setting baseurl to empty would it work.
  (See #16 (comment))
- see #16 (comment) for comments
- fixed _posts where trailing slash was needed
- fixed blog layout where trailing slash in site.blog was needed
@richardjgowers richardjgowers merged commit 549bfb4 into master May 1, 2018
@richardjgowers
Copy link
Member

Looks like it works too!

@orbeckst
Copy link
Member Author

orbeckst commented May 1, 2018

Yes, surprisingly enough.

I am not sure I like the drop down menu; the deprecated https://community.algolia.com/algoliasearch-jekyll-hyde/ creates a full "blog style" page – maybe switch over to that one.

Some results also show up twice, e.g., once as a single blog post and once as part of a paginated blog page. But this is something we can probably fix with a PR against the configuration https://github.com/algolia/docsearch-configs/blob/master/configs/mdanalysis.json ; as I understand it, we can exclude certain paths such as https://www.mdanalysis.org/blog/page8/ <-- https://www.mdanalysis.org/blog/page.*

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

Successfully merging this pull request may close these issues.

add search box
2 participants