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.

Sign up for GitHub

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

Jump to bottom

Add filtering UI to search #130

Closed
ericholscher opened this issue Feb 8, 2023 · 0 comments · Fixed by #132
Closed

Add filtering UI to search #130

ericholscher opened this issue Feb 8, 2023 · 0 comments · Fixed by #132
Assignees
@stsewd

Comments

@ericholscher
Copy link
Member

Our RTD search has a lot of nice filters on it. We should expose this with a a fancier UI element for filters in the extension, along with having them in the query string.

I'm imagining something similar to this:

Screenshot 2023-02-08 at 9 28 58 AM

Having the checkboxes that at least by default includes subprojects.
@ericholscher ericholscher mentioned this issue Feb 15, 2023
Closed
2 tasks
@stsewd stsewd mentioned this issue Mar 9, 2023
Merged
stsewd added a commit that referenced this issue Mar 27, 2023
@stsewd
Use API V3 (#132)
ca36a15 
- Added 2 new options: rtd_sphinx_search_default_filter and rtd_sphinx_search_filters.
- The options are passed as a JSON object to be read by our JS code.
- The default filter defaults to the current project and version (extracted from the RTD env vars)
- Filters are implemented as check boxes, we only support one filter at a time (when a filter is selected, we uncheck all others)

How to test locally:

- Run your local version of RTD
- To allow cross site requests, add `|/api/v3/search` at https://github.com/readthedocs/readthedocs.org/blob/b574f4b0dad34ccba8595cee72246d3473f8a2e5/readthedocs/settings/base.py#L769-L781
- Change the `rtd_dummy_data.json` file
  - `'proxied_api_host': 'http://devthedocs.org'`
  -  Set the project and version to a project that you have locally
- Change the value of `rtd_sphinx_search_default_filter = "project:superproject/latest"` to the slug of a project you have locally
- Build the docs:

```
cd docs
make clean html
```

- Open the _build/index.html file.

### showcase

[showcase.webm](https://user-images.githubusercontent.com/4975310/225148123-7811d477-3a91-4eea-b9fb-b8cb1cbf0d6a.webm)


Closes #129
Closes #130
Closes #131
@humitos humitos mentioned this issue Mar 27, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stsewd stsewd

Labels
None yet
Projects
📍Roadmap
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use API V3 readthedocs/readthedocs-sphinx-search
2 participants
@ericholscher @stsewd