quicksearch breakage in sphinx 1.8.0 and up

[mattip]

Quicksearch is broken on sphinx 1.8.0 and up, see numpy/numpy#12044. It seems the problem is some interaction with the theme's javascript. here is one example of a fix rtfd/sphinx_rtd_theme/pull/672

[pv]

The only javascript is https://github.com/scipy/scipy-sphinx-theme/blob/master/_theme/scipy/static/js/copybutton.js and the copypaste sphinx template in https://github.com/scipy/scipy-sphinx-theme/blob/d990ab9134199f6496b9ac8567f10791f04a720a/_theme/scipy/layout.html#L89-L103

[mattip]

The fix from rtfd changes the first few lines of that codeblock to read options from documentation_options.js instead.

If I build the NumPy documentation locally and go to the quicksearch link build/html/search.html, I can see there is a javascript error:

jQuery.Deferred exception: Search is not defined @file:////numpy/doc/build/html/search.html?q=ASD:60:25
j@file:///numpy/doc/build/html/_static/jquery.js:2:29997
g/</k<@file:///numpy/doc/build/html/_static/jquery.js:2:30313
 undefined

It seems there is more than one thing going on. Replacing the codeblock you posted with the one from the sphinx basic theme does not clear the javascript error

[mattip]

Also see sphinx-doc/sphinx#5460. I am not sure what the upgrade path is to make the theme compliant with the newer version, and what kind of tests exist to make sure it is correct

[rgommers]

and what kind of tests exist to make sure it is correct

Just building the NumPy and SciPy docs and inspecting them; we touch this theme so rarely that it doesn't make sense to write tests for it.

[rgommers]

https://github.com/sphinx-doc/sphinx/issues/5460 says that a fallback was added and this is fixed again. the milestone for the attached PR says in Sphinx 1.8.3

[mattip]

We should still update the theme, but indeed 1.8.3+ is backward compatible.