-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
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
Add autoID for definition terms #13404
Conversation
fcfaa8f
to
0a6878f
Compare
2a6d797
to
412bdd6
Compare
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.
I'll take this for a spin in a bit.
Something to think about for the docs site... since we don't have a |
No. If we want to do this via JS it can be easily be done in "five lines" in a AlpineJS controller. |
This is very slick. Works great with the link render hook on the docs site (e.g., validate And thanks for this too:
|
I have no preference how we do it... this is the last task in eliminating the h6 hack. |
Fixes gohugoio#13403 See gohugoio#11566 Co-authored-by: Joe Mooring <[email protected]>
I added a fix for this in gohugoio/hugoDocs#2914 I like definition lists for documentation and making them as easy to use as possible (by making them easy to link to) is worth it, I guess, so I suggest we go with the approach above. It's simple enough. |
Fixes #13403