Skip to content

Issue #9070: add anchor links to option tables in property_types.xml#19678

Merged
romani merged 1 commit into
checkstyle:masterfrom
ayushactiveat:issue-9070-property-types-anchor-links
Apr 12, 2026
Merged

Issue #9070: add anchor links to option tables in property_types.xml#19678
romani merged 1 commit into
checkstyle:masterfrom
ayushactiveat:issue-9070-property-types-anchor-links

Conversation

@ayushactiveat

@ayushactiveat ayushactiveat commented Apr 11, 2026

Copy link
Copy Markdown
Contributor

fixes #9070
added anchor links to all option table rows in property_types.xml, like used in google_style.xml.

@stoyanK7

Copy link
Copy Markdown
Collaborator

Github, generate site

Comment on lines +217 to +225
<td>
<a name="BlockOption-text"/>
<a href="#BlockOption-text">
<span class="wrapper inline">
<img src="images/anchor.png" alt=""/>
</span>
</a>
<code>text</code>
</td>

@stoyanK7 stoyanK7 Apr 11, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see how we already put anchors. We use JS for that, not HTML: https://github.com/checkstyle/checkstyle/blob/master/src/site/resources/js/anchors.js

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's do same as we did with properties of checks https://checkstyle.org/checks/annotation/annotationlocation.html#Properties

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay

@ayushactiveat ayushactiveat force-pushed the issue-9070-property-types-anchor-links branch from 76392ad to 1f20915 Compare April 11, 2026 23:13
@ayushactiveat ayushactiveat force-pushed the issue-9070-property-types-anchor-links branch from 1f20915 to e04d4f6 Compare April 11, 2026 23:22
@ayushactiveat

Copy link
Copy Markdown
Contributor Author

Github, generate site

@ayushactiveat

Copy link
Copy Markdown
Contributor Author

@romani @stoyanK7 , i did the changes, please review and let me know any other change needed

@stoyanK7 stoyanK7 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Scrolled through the page. Looks good to me👍

@romani romani left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!!

@romani romani merged commit 03125ef into checkstyle:master Apr 12, 2026
123 checks passed
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.

update tables in option page to have link to exact line of table

3 participants