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

Make navigation sidebar sticky using CSS #24

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rikvl
Copy link

@rikvl rikvl commented Jul 21, 2021

Fixes issue #11, but the behaviour for a long ToC (that overflows the element) is a bit different.

@mhvk
Copy link

mhvk commented Aug 6, 2021

@astrofrog - would you, or someone else, be able to review this PR? We're using it in one of my packages now and it is quite nice! @rikvl and I just had a look at the two competing options, and agreed this is the nicest one (so we closed #23).

@stevewhitmore
Copy link

stevewhitmore commented Sep 23, 2021

👍 for this PR. @rikvl mentioned this here and while it may not be a perfect solution it's a step in the right direction. Would it be possible (or a reasonable request) to make the nested bulleted items collapsible? That would solve the overflow problem. I'm asking about this blind and I'm not very familiar with this code base. I came searching after being annoyed all day with having to scroll all the way back to the top in the Python docs.

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 this pull request may close these issues.

3 participants