-
Notifications
You must be signed in to change notification settings - Fork 149
Implemented Heading Permalinks with Hover/Visible Chain Icon #596 #606
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
Conversation
✅ Deploy Preview for bejewelled-pegasus-b0ce81 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
zihanKuang
left a comment
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.
Hi @HIMU-2001 , great work!
It seems you've accidentally mixed in content from your other PR: https://github.com/layer5io/docs/pull/604/files. If you could please remove those changes, we can get this merged.👍
Signed-off-by: HIMANSHU RAI <[email protected]>
783073f to
4bee6fd
Compare
|
Hi @zihanKuang , Apologies for mixing up the contents. |
|
Hi @HIMU-2001 , it seems there's a formatting issue with the tightly grouped headings that have no text in between them. |
Signed-off-by: HIMANSHU RAI <[email protected]>
|
Hi @zihanKuang , I have fixed the formatting issue. |
zihanKuang
left a comment
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.
Great PR! Thanks for your work! @HIMU-2001 👍



Notes for Reviewers
Implemented heading permalinks across all Markdown heading levels using Hugo’s render-heading.html hook. Added a chain icon (🔗) that appears on hover for easy linking, with smooth transitions and consistent sizing for better UX.
This PR fixes #596
Signed commits