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

Docutils 0.17 breaks the captions #367

Closed
jorisvandenbossche opened this issue Apr 3, 2021 · 6 comments · Fixed by #463
Closed

Docutils 0.17 breaks the captions #367

jorisvandenbossche opened this issue Apr 3, 2021 · 6 comments · Fixed by #463

Comments

@jorisvandenbossche
Copy link
Member

Tests on master fail, and the difference is docutils bumped from 0.16 to 0.17. Output of a failing test:

 E       AssertionError: FILES DIFFER:
E       /tmp/pytest-of-runner/pytest-0/test_sidebars_captions0/test_build/test_sidebars_captions.html
E       /tmp/pytest-of-runner/pytest-0/test_sidebars_captions0/test_build/test_sidebars_captions.obtained.html
E       HTML DIFF: /tmp/pytest-of-runner/pytest-0/test_sidebars_captions0/test_build/test_sidebars_captions.obtained.diff.html
E       --- 
E       +++ 
E       @@ -1,6 +1,6 @@
E        <nav aria-label="Main navigation" class="bd-links" id="bd-docs-nav">
E         <div class="bd-toc-item active">
E       -  <p class="caption">
E       +  <p>
E           <span class="caption-text">
E            Section 1
E           </span>

So the "caption" class is gone now. This can also be seen in the unstyled output of those captions in the latest docs (will pin docutils to fix this for now)

@jorisvandenbossche
Copy link
Member Author

Relevant sphinx issue: sphinx-doc/sphinx#9051

@jorisvandenbossche
Copy link
Member Author

As noted in the sphinx issue (sphinx-doc/sphinx#9051), there might also be other CSS broken (eg <div class="section", id=".."> changed into <section id="..">, which might needs different CSS selectors)

@vale981
Copy link

vale981 commented Jul 1, 2021

sphinx-doc/sphinx#9051
appears to be fixed,
can the pin be removed then?

@jorisvandenbossche
Copy link
Member Author

Indeed, thanks for the ping about it.

Although it still depends on the sphinx version you have (but we probably shouldn't care about that?)

@astrojuanlu
Copy link

If I understand correctly, this would still be broken in docutils 0.17 and Sphinx < 4.0.0. However, forcing Sphinx~=4.0 might be premature, since there are still some extensions that are not compatible with it. Just FYI (I don't know what's pydata-sphinx-theme attitude or policy towards dependencies)

@jorisvandenbossche
Copy link
Member Author

It seems that sphinx 3.x added a pin for docutils (sphinx-doc/sphinx#9071), so that's a case we should not worry about

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

Successfully merging a pull request may close this issue.

3 participants