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
Improve some comments related to topicpull XSLT parameters #4176
Improve some comments related to topicpull XSLT parameters #4176
Conversation
413cb00
to
3e0eaab
Compare
3e0eaab
to
9fcb0d2
Compare
<xsl:param name="TABLELINK">NUMBER</xsl:param> | ||
<xsl:param name="FIGURELINK">NUMBER</xsl:param> | ||
<xsl:param name="remove-broken-links" as="xs:string?"/> | ||
<xsl:param name="TABLELINK" as="xs:string" select="'NUMBER'"/> <!-- DITA-OT parameter args.tablelink.style: 'NUMBER' or 'TITLE' --> |
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.
@chrispy-snps I marked this approved because I'm fine with the code comments - but would it be possible to move the code comment to the line above? I worry about a future code linting tool that will move the comment to the next line, and you end up with
<xsl:param name="FIGURELINK" as="xs:string" select="'NUMBER'"/>
<!-- DITA-OT parameter args.figurelink.style: 'NUMBER' or 'TITLE' -->
<xsl:param name="remove-broken-links" as="xs:string" select="'false'"/>
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.
Updated to request that change -- other than that LGTM
9fcb0d2
to
801a7ad
Compare
Signed-off-by: chrispy <chrispy@synopsys.com>
Signed-off-by: Jarno Elovirta <jarno@elovirta.com>
801a7ad
to
3521e2a
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.
LGTM
Description
This pull request adds/updates some parameter-related comments in the topicpullImpl.xsl file.
Motivation and Context
This started when I noticed an outdated comment that a
$tbltitle
template parameter was not used (it is). After fixing that, I also added/updated some other parameter-related comments while tracing through the code.How Has This Been Tested?
All tests pass. I also manually tested
<xref>
and<link>
references to<table>
and<figure>
elements with the parameters set to both values.Type of Changes
These are comments only; no functionality is affected.
Documentation and Compatibility
No documentation or release notes are needed.
Checklist
I tried to follow the format of existing comments in other preprocessing stylesheets.