-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Custom theme isn't showing on rtd to test #518
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
Comments
We ran into this with our project too. Our conclusion was that if it detects the theme name to be 'sphinx_rtd_theme', then it ignores whatever is in your repository and uses their version. If you change the name of the theme to something other than 'sphinx_rtd_theme', then it seems to do something...but its still wrong. Locally, everything looks great. For example, we have:
But in the built docs, we have:
The reason we know it is actually incorporating our changes to the theme is that we can check |
This will be fixed tomorrow when we roll the theme out officially. |
The later issue might not be. We have custom logic that we run when we're on RTD, which is when we don't insert the media files because we host them. If you have customizations forking the theme is currently not the supported way to handle this. Please file bugs against the theme so we can improve it. |
The theme sphinx-rtd-theme open source and forkable. Pip allows targetting packages by forks/branches. Someone could, in theory, have customization for a fork of sphinx-rtd-theme that probably wouldn't be useful to the upstream project. In practice, if it was useful, they'd be PR it anyway to save them the load of maintaining a parallel fork + get the glorious pr ego boost. I may look into this when I finish a few other things. If I start at something I'll mention back here. |
so we're still not able to use custom themes? or am I doing something wrong? http://django-grappelli.readthedocs.org/en/stable-2.5.x/ I added a folder _themes (with a renamed fork of sphinx_rtd_theme) which is part of my repo, the build with RTD works fine but the theme doesn't seem to be loaded. |
Am I correct in inferring that you are interested in supplying your own |
no, I wanna use my own styles. there shouldn´t be a hack involved in using a custom theme, right? |
We have custom logic that we run when we're on RTD, which is when we don't insert the media files because we host them. If you have customizations forking the theme is currently not the supported way to handle this. Please file bugs against the theme so we can improve it. |
It's not bugs: IMHO the current theme can be improved in terms of readability/usability, but I already opened an issue and @snide decided to not take action (which is fine). Now I want to add the updates myself … you're telling me I'm stuck with the theme provided by RTD? No way around this? |
Add an option to configure the media handling, and add it back to the default theme, and use it in your fork? |
good idea — but how can I remove the RTD static files? Is it the line |
it's working, see http://django-grappelli.readthedocs.org/en/stable-2.5.x/ If I find the time, I might add the media handling option … but I can live with this solution for now. thanks four your support. |
Closing this, since if it is still an issue, its a problem with the theme and not RTD. |
…yout.html It seems that readthedocs.org basically overwrites any customization you may have through layout.html: readthedocs/readthedocs.org#518 . My impression is that you can do almost as well by overriding page.html: it looks like it doesn't get processed for every page, but it kicks in for most of them, and RTD doesn't override that one. This commit is a test to see if this works on the website. Since I haven't gotten rid of layout.html, locally-generated docs have two CSS includes of pwkit.css. That seems fine for now.
After reading thru readthedocs/readthedocs.org#518 and experimenting, this seems to be the way to get the syntax diagram style sheets right on readthedocs.org
I am not sure if this is my config, the sphinx_rtd_theme package or readthedocs.org - but I can't get my branch of the theme - as a package or a submodule, to show on readthedocs.org.
https://github.com/tony/sphinx_rtd_theme/tree/sphinx-basic-updates (see doc/_rtd_theme)
https://github.com/tony/tmuxp/blob/rtd_theme_sphinx-basic-updates/doc/conf.py
https://readthedocs.org/builds/tmuxp/930992/
http://tmuxp.readthedocs.org/en/rtd_theme_sphinx-basic-updates/ - just shows normal rtd theme.
The text was updated successfully, but these errors were encountered: