sphinx-book-theme icon indicating copy to clipboard operation
sphinx-book-theme copied to clipboard

Wrong content section in the sidebar

Open mathbunnyru opened this issue 2 years ago • 11 comments

Describe the bug

context I open https://sphinx-book-theme.readthedocs.io/en/stable/tutorials/get-started.html#add-a-source-repository-button-to-your-theme or press "Add a source repository button to your theme" content button on https://sphinx-book-theme.readthedocs.io/en/stable/tutorials/get-started.html.

expectation I expected content section on the right panel to change to "Add a source repository button to your theme".

bug But instead it tells me that the previous section is active.

If I scroll down a little bit, then the section is correct.

Снимок экрана 2022-03-29 в 17 14 56

Reproduce the bug

  1. Go to https://sphinx-book-theme.readthedocs.io/en/stable/tutorials/get-started.html#add-a-source-repository-button-to-your-theme

List your environment

OS: macOS 12.3 Browser: Safari 15.4

mathbunnyru avatar Mar 29 '22 16:03 mathbunnyru

I am noticing this too with v0.3.2. Have you found any way to resolve this @mathbunnyru?

haklc avatar May 20 '22 09:05 haklc

No, I haven’t. But I didn’t actually try to find it, because web is definitely not my area of expertise.

mathbunnyru avatar May 20 '22 09:05 mathbunnyru

It seems that the active class tag is being added to the wrong <li> entry in the right side menu (the entry above the actually active one).

Was this present in any of the previous versions? I just started using this theme.

haklc avatar May 26 '22 08:05 haklc

Clicking on one of the entries in the menu positions you exactly 12 pixels above the chosen section.

haklc avatar May 26 '22 09:05 haklc

Currently, in the body tag of each HTML file the Bootstrap Scrollspy data-offset parameter is set to "60". By changing this to "73" it works as expected, but there is some jittering. The optimal value seems to be "80".

haklc avatar May 26 '22 10:05 haklc

I just started using this theme.

Me as well, we've only started using it quite recently and the old versions don't work well for us for another reason.

@haklc could you make a PR changing this value of Bootstrap Scrollspy?

@choldgraf may I ask you to take a look here?

mathbunnyru avatar May 27 '22 18:05 mathbunnyru

This is now fixed upstream, we just need to wait for a release then update here.

choldgraf avatar May 27 '22 18:05 choldgraf

Great, thank you!

mathbunnyru avatar May 27 '22 18:05 mathbunnyru

When is the next release expected? I'd love to use this theme but I cannot manually fix this value after HTML files are generated on Read the Docs.

haklc avatar May 30 '22 09:05 haklc

There's a release candidate for the upstream theme out now (https://github.com/pydata/pydata-sphinx-theme/releases/tag/v0.9.0rc1) and then it's a question of doing the necessary changes to make this theme support it. No promises on timeline though, but hopefully it shouldn't be too long.

choldgraf avatar May 30 '22 11:05 choldgraf

The upstream new version has been released 🎉 https://github.com/pydata/pydata-sphinx-theme/releases/tag/v0.9.0

mathbunnyru avatar Jun 12 '22 08:06 mathbunnyru

Any news as to when a newer version of pydata-sphinx-theme will be integrated into sphinx-book-theme?

ClaudiaSchulz avatar Dec 19 '22 13:12 ClaudiaSchulz

Yep this will be closed by the following PR and we'll get a release candidate out soon after that:

  • https://github.com/executablebooks/sphinx-book-theme/pull/640

choldgraf avatar Dec 19 '22 14:12 choldgraf

I think this works much better now, but subtopic links are a bit broken. https://jupyter-docker-stacks--1855.org.readthedocs.build/en/1855/#jupyter-notebook-deprecation-notice

When I visit this page, it doesn't show Jupyter Notebook Deprecation Notice subtopic (and I think it should). It shows only the text under it. Maybe it doesn't consider there is a topbar? Screenshot 2023-01-05 at 15 50 11

I've used just released rc version: https://github.com/jupyter/docker-stacks/pull/1855

mathbunnyru avatar Jan 05 '23 11:01 mathbunnyru

good catch - fixing it in:

  • #669

choldgraf avatar Jan 05 '23 13:01 choldgraf