RFR: 8322708: Global HTML attributes are not allowed
Chen Liang
liach at openjdk.org
Tue Jun 11 11:55:13 UTC 2024
On Tue, 11 Jun 2024 11:37:27 GMT, Nizar Benalla <nbenalla at openjdk.org> wrote:
> Can I please get a review for this change, that aims to add support for Global HTML tags.
> Here is the [link](https://cr.openjdk.org/~nbenalla/javadocGlobalAttrs/pkg1/package-summary.html) to the generated docs.
> Thanks in advance.
src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclint/Checker.java line 694:
> 692: // without checking the validity or applicability of the name
> 693: // custom "data-*" attributes are also accepted
> 694: var attrName=name.toString();
Suggestion:
var attrName = name.toString();
src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclint/HtmlTag.java line 513:
> 511: VSPACE,
> 512: WIDTH,
> 513: ACCESSKEY,
Maybe add a comment `// Global attributes` above this line to indicate the start of global attributes, so future attributes are added to the correct section?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19652#discussion_r1634727172
PR Review Comment: https://git.openjdk.org/jdk/pull/19652#discussion_r1634727032
More information about the javadoc-dev
mailing list