Integrated: 8336259: Wrong link to stylesheet.css in JavaDoc API documentation
Nizar Benalla
nbenalla at openjdk.org
Fri Jul 12 21:53:55 UTC 2024
On Thu, 11 Jul 2024 20:55:29 GMT, Nizar Benalla <nbenalla at openjdk.org> wrote:
> Can I please get a review for this small change, the relative link to the stylesheet isn't needed as it wasn't used anyway in the generated HTML. The correct link to the stylesheet is already in the generated HTML.
>
> This is the difference between the generated docs before and after this change, by running `diff -r` on the docs before and after the change.
>
>
> GeneratedDocs % diff -r 8336259-stylesheet old-docs
> diff -r 8336259-stylesheet/api/java.base/java/lang/doc-files/ValueBased.html old-docs/api/java.base/java/lang/doc-files/ValueBased.html
> 13a14
>> <link rel="stylesheet" type="text/css" href="../../../../stylesheet.css" title="Style">
> diff -r 8336259-stylesheet/api/java.base/java/lang/doc-files/threadPrimitiveDeprecation.html old-docs/api/java.base/java/lang/doc-files/threadPrimitiveDeprecation.html
> 13a14
>> <link rel="stylesheet" type="text/css" href="../../../../stylesheet.css" title="Style">
>
>
> Here is the link for the generated docs after the change, no visual change is seen but the erronous link is gone.
> Value based classes [before](https://docs.oracle.com/en%2Fjava%2Fjavase%2F22%2Fdocs%2Fapi%2F%2F/java.base/java/lang/doc-files/ValueBased.html) - [after](https://cr.openjdk.org/~nbenalla/GeneratedDocs/8336259-stylesheet/api/java.base/java/lang/doc-files/ValueBased.html)
>
> Java Thread Primitive Deprecation [before](https://docs.oracle.com/en%2Fjava%2Fjavase%2F22%2Fdocs%2Fapi%2F%2F/java.base/java/lang/doc-files/threadPrimitiveDeprecation.html) - [after](https://cr.openjdk.org/~nbenalla/GeneratedDocs/8336259-stylesheet/api/java.base/java/lang/doc-files/threadPrimitiveDeprecation.html)
This pull request has now been integrated.
Changeset: 5bc86f33
Author: Nizar Benalla <nbenalla at openjdk.org>
Committer: Chen Liang <liach at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/5bc86f332986e3fffc1363f569029bb73a706064
Stats: 4 lines in 2 files changed: 0 ins; 2 del; 2 mod
8336259: Wrong link to stylesheet.css in JavaDoc API documentation
Reviewed-by: jjg, liach
-------------
PR: https://git.openjdk.org/jdk/pull/20145
More information about the core-libs-dev
mailing list