On Fri, 9 Oct 2026 11:37:02 GMT, Hannes Wallnöfer <[email protected]> wrote:

>> Please review a change to solve a number of issues in the HTML presentation 
>> of the JVM Tool Interface specification that prevented it from using 
>> JDK-level documentation features such as CSS themes and heading permalinks:
>> 
>>  - Use wrapped `<section>` elements instead of empty `<p>` elements  for 
>> section ids
>>  - Use the CSS `light-dark()` function for colors in inline styles to 
>> support the dark theme
>>  - Resolve a number of unwanted interactions of the inline styles with the 
>> `jdk-default.css` stylesheet
>>  - Remove the need for hard-coded `<div class="sep"/>` separators by 
>> defining proper margins
>>  - Move `<hr>` tags between sections and don't use them for section `id` 
>> attributes
>>  - Solve minor whitespace issue by setting `indent="no"` on `<xsl:output>`
>> 
>> The rendered `jvmti.html` file as processed by the JDK build script [can be 
>> viewed 
>> here](https://cr.openjdk.org/~hannesw/8393263/docs.00/specs/jvmti.html). It 
>> now supports light and dark themes, hovering over headings shows a permalink 
>> icon, and various rendering problems resulting from interaction with 
>> `jdk-default.css` (especially in function/callback tables) have been fixed. 
>> The spec is also easier to read on very wide and very small displays.
>> 
>> The unprocessed version of the spec as generated by the XSL stylesheet [is 
>> also available](https://cr.openjdk.org/~hannesw/8393263/raw.00/jvmti.html). 
>> It should be visually unchanged from the old version.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> 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 10 commits:
> 
>  - Merge branch master
>  - Update build to dedicated specs script.
>  - Merge branch 'JDK-8377548' into JDK-8393263
>  - Add dedicated script and link icon for other specs and tool guides
>  - Restore underlined link default behavior
>  - 8393263: JVM Tool Interface spec presentation needs work
>  - Refine scroll margin
>  - Improve heading permalink code
>  - Add color-scheme CSS property to enable light-dark() in inline styles
>  - 8377548: Implement dark theme for other specs and tool guides

Change looks great overall!
There no permalink for this 
[title](https://cr.openjdk.org/~hannesw/8393263/docs.00/specs/jvmti.html#ChangeHistory)
 though.
I think there is a small issue with the HTML

-------------

PR Comment: https://git.openjdk.org/jdk/pull/33181#issuecomment-6081740650

Reply via email to