Skip to content

Fix links that no longer jump to their headings after the HonKit migration - #705

Merged
kenhys merged 1 commit into
fluent:1.0from
Watson1978:fix-honkit-anchor-links
Oct 7, 2026
Merged

kenhys merged 1 commit into
fluent:1.0from
Watson1978:fix-honkit-anchor-links

Conversation

@Watson1978

Copy link
Copy Markdown
Contributor

HonKit builds heading ids with different rules from GitBook. It drops _, <, > and /, and does not add GitBook's id- prefix. Because of this, 26 links in 16 pages still used the GitBook ids and opened the page without scrolling to the heading.

This updates those anchors to the ids in the HonKit output. For example, system-config.md#file_permission becomes system-config.md#filepermission. Only the anchors change. After a rebuild, every internal link in the page bodies resolves to an existing page and id.

🤖 Generated with Claude Code

HonKit builds heading ids with different rules from GitBook. It drops
"_", "<", ">" and "/", and does not add the "id-" prefix. 26 links
still used the GitBook ids, so they did not jump to the heading.
Update them to the ids in the generated HTML.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
@netlify

netlify Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for fluentd-docs ready!

Name Link
🔨 Latest commit 5ad7aff
🔍 Latest deploy log https://app.netlify.com/projects/fluentd-docs/deploys/6ac5d0257f12f800073edaea
😎 Deploy Preview https://deploy-preview-705--fluentd-docs.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.

@Watson1978
Watson1978 requested a review from kenhys October 7, 2026 04:56
@kenhys
kenhys merged commit fb109e8 into fluent:1.0 Oct 7, 2026
6 checks passed
@Watson1978
Watson1978 deleted the fix-honkit-anchor-links branch October 7, 2026 08:27
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