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

html_baseurl and READTHEDOCS variables #259

Open
ru-fu opened this issue Jul 17, 2024 · 1 comment
Open

html_baseurl and READTHEDOCS variables #259

ru-fu opened this issue Jul 17, 2024 · 1 comment

Comments

@ru-fu
Copy link
Contributor

ru-fu commented Jul 17, 2024

See https://about.readthedocs.com/blog/2024/07/addons-by-default/

The html_baseurl and READTHEDOCS variables will be deprecated. We don't use them in the starter pack, so most likely, we are not affected.
However, several of the packages we install use them (mainly Furo and OpenGraph), so we'll need to verify that these packages include the required changes. If they don't, we'll need to include the code from the blog post in the starter pack.

@IronCore864
Copy link
Contributor

IronCore864 commented Oct 8, 2024

It seems this change is necessary, otherwise the ad placement would be wrong:

If the left-side navigation bar is not very long, the ad should show up on the left; if the nav bar is very long, taking up the entire screen height, the ad should show up on the bottom right as a pop-up.

Take RTD's own doc site as an example, if you visit this page where the nav bar isn't too long, the ad shows up on the left, but if you access this one, since the nav bar takes up all the space, the ad shows up as a pop-up on the bottom right.

After Oct 7, in order for this to work as expected, the change mentioned in the blog is required, but adding the READTHEDOCS variable alone isn't sufficient, because the furo theme uses more variables if READTHEDOCS is set, for example here. So, everything defined here should also be set in the starter pack, and the docs for the starter pack also need to be updated to remind the users to adjust these values accordingly. See a PR fixing this issue in another repo using the starter pack here.

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

No branches or pull requests

2 participants