RFR: JDK-8287597: List all preview features on the javadoc PREVIEW page [v2]
Hannes Wallnöfer
hannesw at openjdk.org
Thu Sep 22 17:28:26 UTC 2022
On Thu, 18 Aug 2022 21:42:04 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:
>> Hannes Wallnöfer has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains two commits:
>>
>> - Merge branch 'master' into JDK-8287597
>> - JDK-8287597: List all preview features on the javadoc PREVIEW page
>
> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/SummaryListWriter.java line 164:
>
>> 162: protected void addIndexLink(HtmlId id, String headingKey, Content content) {
>> 163: var li = HtmlTree.LI(links.createLink(id,
>> 164: contents.getContent(headingKey))).setId(HtmlId.of("contents-" + id.name()));
>
> Suggest adding a comment that `"contents-" + id` appears in the JavaScript code as well. You presumably can't change one without the other (and can't use a common function across the language barrier.)
Excellent point.
-------------
PR: https://git.openjdk.org/jdk/pull/9336
More information about the core-libs-dev
mailing list