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

Add table of contents. #64

Merged
merged 4 commits into from
May 5, 2022
Merged

Add table of contents. #64

merged 4 commits into from
May 5, 2022

Conversation

lberki
Copy link
Contributor

@lberki lberki commented Mar 25, 2022

Only tested on a Chromebook and Firefox on Android.

Lukacs T. Berki added 4 commits March 25, 2022 04:09
It can be enabled using the `toc: true` field in the frontmatter.

It auto-expands on hover. If JS is enabled, the expansion is
animated. The animation is apparently impossible to implement
without JS :(

On mobile, expands and unexpands on tap instead of hover.
Why this works is a mystery, but it sure does.
@EmielH
Copy link
Owner

EmielH commented Mar 29, 2022

Thanks for your contribution! It's a large one 😄 I had a quick look this weekend, will try to merge soon!

@lberki
Copy link
Contributor Author

lberki commented May 4, 2022

(friendly ping)

@EmielH EmielH merged commit c95051b into EmielH:master May 5, 2022
@EmielH
Copy link
Owner

EmielH commented May 5, 2022

Thanks for the ping. My apologies - I've had a busy couple of weeks!

@lberki
Copy link
Contributor Author

lberki commented May 6, 2022

Thanks! I can relate to the busy couple of weeks part :)

TBH I was expecting a long and meticulous code review given my experience in frontend development (or lack thereof), but if I managed to clear the quality bar on the first go, all the better.

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.

2 participants