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

Docs search bar doesn't work #1125

Closed
siranipour opened this issue Mar 3, 2021 · 7 comments
Closed

Docs search bar doesn't work #1125

siranipour opened this issue Mar 3, 2021 · 7 comments
Assignees

Comments

@siranipour
Copy link
Contributor

Currently it just says searching for ages

@RosalynLP
Copy link
Contributor

RosalynLP commented Mar 3, 2021

sphinx-doc/sphinx#3203 I think this is what's happening...?
Although this is a v old issue so maybe not - our version is 3.0.3 in any case and it did use to work for us

@RosalynLP RosalynLP reopened this Mar 3, 2021
@RosalynLP
Copy link
Contributor

whoops butterfingers...

@RosalynLP RosalynLP mentioned this issue Mar 3, 2021
33 tasks
@Zaharid
Copy link
Contributor

Zaharid commented Mar 3, 2021

This is some js bug from sphinx

Uncaught ReferenceError: Stemmer is not defined
    query https://docs.nnpdf.science/_static/searchtools.js:158
    setIndex https://docs.nnpdf.science/_static/searchtools.js:98
    <anonymous> https://docs.nnpdf.science/search.html?q=njnj&check_keywords=yes&area=default line 2 > injectedScript:1
    jQuery 8
    loadIndex https://docs.nnpdf.science/_static/searchtools.js:84
    <anonymous> https://docs.nnpdf.science/search.html?q=njnj&check_keywords=yes&area=default:206
    jQuery 13
searchtools.js:158:19

maybe we can just use some version that works?

@Zaharid
Copy link
Contributor

Zaharid commented Mar 3, 2021

sphinx-doc/sphinx#8623

@siranipour
Copy link
Contributor Author

Ok it's because of our theme. pip install --upgrade sphinx-rtd-theme fixes it, nice and easy fix

@Zaharid
Copy link
Contributor

Zaharid commented Mar 4, 2021

Just sticking sphinx_rtd_theme>0.5 in the conda recipe should fix it.

@RosalynLP
Copy link
Contributor

yeah that works, I'll add it in #1122

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

No branches or pull requests

3 participants