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

DOC: fix the inclusion of the custom css file #2858

Merged
merged 1 commit into from
Sep 17, 2021

Conversation

jorisvandenbossche
Copy link
Contributor

See mailing list message about broken css of proj.org. It seems that the readthedoc theme's css was no longer included in the <head> (only the css overrides of proj itself). Might be triggered due to changes in sphinx_rtd_theme 1.0, which was just released, but didn't verify that.

I think this patch fixes that (because sphinx puts the parent theme's css in css_files, and thus the conf.py overrides this. And starting with sphinx_rtd_theme, it only relies on that readthedocs/sphinx_rtd_theme@836467e), and in any case it now includes the custom css as documented by rtd (https://docs.readthedocs.io/en/stable/guides/adding-custom-css.html)

@kbevers
Copy link
Member

kbevers commented Sep 17, 2021

Let's see if it works

@kbevers kbevers merged commit f22f302 into OSGeo:master Sep 17, 2021
@mwtoews
Copy link
Member

mwtoews commented Sep 17, 2021

Artefact looks good, thanks @jorisvandenbossche !

@kbevers
Copy link
Member

kbevers commented Sep 17, 2021

Yeah, and I've backported it to 8.1 manually. Thanks for fixing this!

@OSGeo OSGeo deleted a comment from github-actions bot Jan 13, 2023
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.

None yet

3 participants