Skip to content

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

Merged
merged 2 commits into from
Jun 16, 2025

Conversation

HIMU-2001
Copy link
Contributor

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

  • Yes, I signed my commits.

Copy link

netlify bot commented Jun 16, 2025

Deploy Preview for bejewelled-pegasus-b0ce81 ready!

Name Link
🔨 Latest commit a9815dc
🔍 Latest deploy log https://app.netlify.com/projects/bejewelled-pegasus-b0ce81/deploys/6850a319daf42600084961d6
😎 Deploy Preview https://deploy-preview-606--bejewelled-pegasus-b0ce81.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

@zihanKuang zihanKuang left a 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 <himanshuuu.2001@gmail.com>
@HIMU-2001 HIMU-2001 force-pushed the heading-permalinks branch from 783073f to 4bee6fd Compare June 16, 2025 22:32
@HIMU-2001
Copy link
Contributor Author

Hi @zihanKuang ,

Apologies for mixing up the contents.
I have removed the unecessary commits/file modification from this PR.

@HIMU-2001 HIMU-2001 closed this Jun 16, 2025
@HIMU-2001 HIMU-2001 reopened this Jun 16, 2025
@zihanKuang
Copy link
Contributor

Hi @HIMU-2001 , it seems there's a formatting issue with the tightly grouped headings that have no text in between them.

image

original:
image

Signed-off-by: HIMANSHU RAI <himanshuuu.2001@gmail.com>
@HIMU-2001
Copy link
Contributor Author

Hi @zihanKuang ,

I have fixed the formatting issue.

Current state :
image

Copy link
Contributor

@zihanKuang zihanKuang left a 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 👍

@zihanKuang zihanKuang merged commit 984a607 into layer5io:master Jun 16, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: Implement Heading Permalinks with Hover/Visible Chain Icon
2 participants