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

sidebar directive in the new doc theme not producing a sidebar #9031

Closed
hoechenberger opened this issue Mar 12, 2021 · 4 comments
Closed

sidebar directive in the new doc theme not producing a sidebar #9031

hoechenberger opened this issue Mar 12, 2021 · 4 comments
Assignees

Comments

@hoechenberger
Copy link
Member

It appears .. sidebar:: will not actually create a sidebar anymore – which does make sense IMHO with the new layout; however maybe we should replace "sidebars" with "info boxes" now?

Source:
Screen Shot 2021-03-12 at 12 24 06

Rendering:
Screen Shot 2021-03-12 at 12 23 42

@larsoner
Copy link
Member

@larsoner
Copy link
Member

... actually we shouldn't even really use that anymore since it's for sphinx-bootstrap-theme, which uses BS3, and pydata-sphinx-theme uses BS4 I think.

I'd propose adding whatever the correct thing is (if it is info boxes) to the upstream pydata-sphinx-theme to see if they'd want it. If they do then let's implement it there, if they don't then we can vendor in our own doc/sphinxext/* somewhere.

drammock added a commit to drammock/mne-python that referenced this issue Mar 13, 2021
@drammock drammock self-assigned this Mar 13, 2021
@drammock
Copy link
Member

actually we shouldn't even really use that anymore since it's for sphinx-bootstrap-theme, which uses BS3, and pydata-sphinx-theme uses BS4 I think.

FYI I updated doc/sphinxext/sphinx_bootstrap_divs when we switched themes, to be BS4 compliant. (not certain I did it fully or in the best way, but I at least made it work).

The real problem here was that the sidebars used to be put in a div and now they're in an aside tag instead, and our CSS selector was too specific. I've pushed a fix as part of #9036.

@hoechenberger
Copy link
Member Author

Thanks @drammock!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants