-
Notifications
You must be signed in to change notification settings - Fork 201
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
Set "readthedocs" ad placement explicitly #169
Comments
While your mentioning RTD, I would also link pydata/pydata-sphinx-theme#194 |
@chrisjsewell I've actually open an issue for this very purpose: pydata/pydata-sphinx-theme#345 I'm just jumping here, Ware there any evolution on this matter since August ? |
@12rambau nope - we'd still like it to happen - if anyone would like to give a shot at a PR I am happy to review! |
I was looking on how you handling the ad placement in RDT and Google lead me here but I think I'm actually off-topic
If only, I was good enough in HTML/CSS 😄 |
I think I found a good pattern here: https://github.com/pradyunsg/furo/blob/main/src/furo/theme/furo/sidebar/ethical-ads.html?rgh-link-date=2021-03-20T19%3A15%3A02Z updated the top comment |
Right now, if people host their documentation on readthedocs, it will automatically put an advertisement at the bottom of the page:
It's not too obtrusive, though it clashes a bit with our CSS and overlaps with some elements.
I believe that we can include some HTML that will instead tell RTD to put the add there instead of at the bottom, and so we can control how it looks etc a bit better. What do folks think about that?
I believe we need to add: https://github.com/pradyunsg/furo/blob/main/src/furo/theme/furo/sidebar/ethical-ads.html?rgh-link-date=2021-03-20T19%3A15%3A02Z
The text was updated successfully, but these errors were encountered: