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

Custom nbsphinx-gallery.css does not take into account - sphinx 7.2.6 #778

Open
ThibaultFy opened this issue Feb 22, 2024 · 7 comments · Fixed by Substra/substra-documentation#394

Comments

@ThibaultFy
Copy link

ThibaultFy commented Feb 22, 2024

Hello there,

When upgrading to Sphinx 7.2.6, my custom nbsphinx-gallery.css is not taken into account anymore (it is overwrite by a basic one).

I tried to explicitely add it to html_css_files -> same result.

Do you have any idea ?

Thanks !

@mgeier
Copy link
Member

mgeier commented Feb 22, 2024

No, I have no idea, it sounds like a bug.

Can you bisect this?

@ThibaultFy
Copy link
Author

ThibaultFy commented Feb 23, 2024

Thanks for the quick answer.

Not much to bisect... I had the custom css nbsphinx-gallery.css working on a previous Sphinx version and nbsphinx 9.3 following doc requirements. But updgrading to Sphinx 7.2.6 broke it.

The conf.py is this one (pointing to the relevant line).

To give you an example, here the gallery with the custom CSS, and here with the bug.

@mgeier
Copy link
Member

mgeier commented Feb 24, 2024

Not much to bisect...

Maybe we are using different interpretations of the term "bisect".

I wanted to suggest using git bisect. The result of this would be a single Git commit, which would be the one introducing the bug. With this, we can then ask the Sphinx team to fix it (or maybe it turns out to be something to be fixed in nbsphinx).

@ThibaultFy
Copy link
Author

Oh okay sorry I didn't understood it this way. But which repo do you want me to bisect ? On my side, it is this one that introduced the bug, when bumping from Sphinx 4.4.0 to Sphinx 7.2.6.

@ThibaultFy
Copy link
Author

I successfully fixed it by:

  • renaming the folder static to _static
  • add the css file to html_css_files var (e.g html_css_files = ["my-nbsphinx-gallery.css"]
  • the custom css can't be name nbsphinx-gallery.css anymore, or it is overwritten by the default one.

@mgeier
Copy link
Member

mgeier commented Mar 6, 2024

It's great that you found a work-around, but this doesn't seem to be a real fix for the problem.

renaming the folder static to _static

You should be able to choose the folder name arbitrarily, you just have to use the same name in html_static_path.

add the css file to html_css_files var (e.g html_css_files = ["my-nbsphinx-gallery.css"]

If you use your own additional file name, you should do that, but if you want to overwrite an existing file name, you should not have to add it there.

the custom css can't be name nbsphinx-gallery.css anymore, or it is overwritten by the default one.

Yeah, but that's the bug, you should be able to overwrite it.

But which repo do you want me to bisect ?

https://github.com/sphinx-doc/sphinx/, because that's where we suspect the bug, right?

@mgeier
Copy link
Member

mgeier commented Mar 14, 2024

I have bisected it and created sphinx-doc/sphinx#12096.

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.

2 participants