-
-
Notifications
You must be signed in to change notification settings - Fork 115
feat: Add data-skip-inventory
boolean attribute for elements to skip local inventory
#774
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
Great, tests broke with default |
The initial suggestion was that we would use a However, that has approach 2 problems:
Therefore, I suggest that we only add <h1 data-skip-inventory></h1>
<h1 data-skip-inventory=""></h1>
<h1 data-skip-inventory="true"></h1>
<h1 data-skip-inventory="false"></h1> <!-- sic --> would be treated as excluded from registration in the inventory, and only elements without the presence of |
data-skip-inventory
boolean attribute for elements to skip local inventory
Check https://smokeshow.helpmanual.io/0a4t3q1d6l2f062y3b67/usage/configuration/headings/#skip_local_inventory from mkdocstrings/python#297 to see how this works in practice :) |
Thanks for the analysis and taking the right direction! Also, clever example in the docs! |
Co-authored-by: Timothée Mazzucotelli <dev@pawamoy.fr>
Co-authored-by: Timothée Mazzucotelli <dev@pawamoy.fr>
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.
OK LGTM, thanks a lot!
Closes #671