-
Notifications
You must be signed in to change notification settings - Fork 7
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
Sticky toc #44
base: master
Are you sure you want to change the base?
Sticky toc #44
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this! It looks good and is a nice QOL improvement. One thing to fix: on pages with long TOCs (e.g. hosting guidelines), the TOC extends off the page and the TOC sidebar is not scrollable.
@ryanrosenberg good catch! i thought it might be a problem but was hoping we didn't have any long enough TOCs :'( i'll do some research on how best to handle it, though scrollable is probably the way to go |
❌ Deploy Preview for gorgeous-parfait-f2be7f failed.
|
pages with table of contents now have them as fixed sidebars :-)
TODO