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 anchor to each parameter row #66895
Add anchor to each parameter row #66895
Conversation
Co-Authored-By: Felix Fontein <felix@fontein.de>
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.
LGTM
Hmm.. I generated this locally (make webdocs off this branch) and I'm not seeing any additional anchors on the parameters of a module... |
docs/templates/plugin.rst.j2
Outdated
@@ -106,7 +106,7 @@ Parameters | |||
<th width="100%">Comments</th> | |||
</tr> | |||
{% for key, value in options|dictsort recursive %} | |||
<tr> | |||
<tr id="parameter-@{ key }@"> |
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.
There's one downside: this will only work well for top-level options. Nested options might create collisions since their ID doesn't contain the keys of the options in which they are nested.
I've added two commits to this PR:
|
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.
LGTM
@dnmvisser thanks for getting this started! |
* Add anchor to each paramater row * Update docs/templates/plugin.rst.j2 Co-Authored-By: Felix Fontein <felix@fontein.de> * Insert full keys into plugin docs. * Added visible links. Co-authored-by: Felix Fontein <felix@fontein.de> (cherry picked from commit 5b93a14)
* Add anchor to each paramater row * Update docs/templates/plugin.rst.j2 * Insert full keys into plugin docs. * Added visible links. Co-authored-by: Felix Fontein <felix@fontein.de> (cherry picked from commit 5b93a14) Co-authored-by: Dick Visser <dick.visser@geant.org>
Addresses #66893