Skip to content

Comments

feat(blog): add table of contents#223

Open
WalkQuackBack wants to merge 10 commits intocatppuccin:mainfrom
WalkQuackBack:feat/blog-toc
Open

feat(blog): add table of contents#223
WalkQuackBack wants to merge 10 commits intocatppuccin:mainfrom
WalkQuackBack:feat/blog-toc

Conversation

@WalkQuackBack
Copy link
Member

Table of contents sidebar on blog pages. Draft as mobile handling is WIP.

Table of contents is off to side of the prose on large screens, and under a navigation button on smaller screens.

@WalkQuackBack
Copy link
Member Author

I got a email about this regarding the check failing due to a missing token: is there anything I have to do on my end to resolve this?

@sgoudham
Copy link
Contributor

Sorry, I forgot about this for forks. That workflow should not try to deploy if it's ran from a fork (which is slightly annoying)

Nothing for you to do.

@WalkQuackBack WalkQuackBack marked this pull request as ready for review April 18, 2025 19:43
@sgoudham sgoudham linked an issue Apr 19, 2025 that may be closed by this pull request
Copy link
Member

@unseen-ninja unseen-ninja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cheers; This is a lovely addition! Tho I've notices that the grid starts to behave a little odd the larger the screen resolution gets. The actual content shrinks more and more while the ToC inhabits most of the space – but that should be the other way around.

Screen.Recording.2025-04-23.at.11.18.38.mov

The ToC's headline is an h2 which I'm not entirely sure about. I would probably set it to an h3 but that might be nonsense semantically? Either way, making it smaller visually might already help to make the initial view feel a little less cluttered.

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.

Add ToC to Blog

3 participants