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
agogo theme: Wrong use of sidebar* configs #10520
Conversation
sphinx/sphinx/themes/agogo/static/agogo.css_t Lines 189 to 195 in 03aaed9
sphinx/sphinx/themes/agogo/static/agogo.css_t Lines 273 to 283 in a950df1
Has this been wrong for that long? A |
Apparently. But I guess the And the wrong sidebar width might also be barely noticeable. I didn't notice that on a rendered page (does anyone even use |
That's in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with your logic, this CSS is currently incorrect. The theme options page notes:
- sidebarwidth (CSS length): Width of the sidebar, default 20em.
- rightsidebar (true or false): Put the sidebar on the right side. Defaults to True.
It is implicit, but reasonably clear, that it is talking about the gutter sidebar rather than the reST .. sidebar::
directive.
A
I wrote above:
I was wrong, because sphinx/sphinx/themes/agogo/theme.conf Line 11 in af425c0
See also https://sphinx-themes.org/sample-sites/default-agogo/. So the And with my suggested change, it stays right, even if |
There are two different things called "sidebar":
sphinxsidebar
sidebar
directive, CSS classsidebar
The config values
sidebarwidth
andrightsidebar
are supposed to control the former.Feature or Bugfix