Skip to content

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.

@netlify
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.👍

@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 <[email protected]>
@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