pydata / xarray

N-D labeled arrays and datasets in Python
https://xarray.dev
Apache License 2.0
3.61k stars 1.08k forks source link

Docs sidebar is squished #8619

Open TomNicholas opened 9 months ago

TomNicholas commented 9 months ago

What happened?

Since the v2024.01.0 release yesterday, there seems to be a rendering error in the website - the sidebar is squished up to the left:

Screenshot 2024-01-18 at 9 50 33 AM
jhamman commented 9 months ago

this may be the same issue as here: https://github.com/pydata/xarray/issues/8366

TomNicholas commented 9 months ago

This doesn't happen when viewing the docs for the previous release, so something must have changed...

https://docs.xarray.dev/en/v2023.11.0/

TomNicholas commented 9 months ago

This is a bit of a problem - the squishing often makes the sidebar unreadable for me now.

@andersy005 wondering if you have any ideas what might have caused this?

dcherian commented 9 months ago

Probably an upstream bug: https://github.com/pydata/pydata-sphinx-theme

TomNicholas commented 9 months ago

Are we actually using pydata-sphinx-theme? From our docs env it looks like we are just using sphinx-book-theme?

https://github.com/pydata/xarray/blob/35b7ab1c5158382336cfb631251b6da84619fe27/ci/requirements/doc.yml#L35

kmuehlbauer commented 9 months ago

@TomNicholas AFAIK, sphinx-book-theme depends on pydata-sphinx-theme.

cdeil commented 9 months ago

The browser console shows this pydata-sphinx-theme.js error:

Uncaught TypeError: Cannot read properties of null (reading 'classList') at HTMLDocument.T (pydata-sphinx-theme.js:570:5)

Probably needs someone familiar with your docs and https://github.com/pydata/pydata-sphinx-theme to track down the root cause and possible fix.

Screenshot 2024-01-20 at 20 58 27
keewis commented 9 months ago

I think the reason is the upgrade of sphinx-book-theme from 1.0.1 to 1.1.0. Not sure what exactly the cause is, but we can work around this by pinning sphinx-book-theme for now.

Edit: funnily enough, pint, which basically copied and adapted our setup, doesn't appear to have this issue.

TomNicholas commented 9 months ago

Whatever the cause of the problem, pinning sphinx-book-theme to 1.0.1 seems to fix it temporarily (see #8632), so I will merge that and re-release, but leave this issue open until we can un-pin.

headtr1ck commented 3 months ago

Is this fixed now? For me it seems to work properly.