RFR: JDK-8261499: Simplify HTML for javadoc links [v2]
Hannes Wallnöfer
hannesw at openjdk.java.net
Thu Feb 11 16:03:10 UTC 2021
> This change gets rid of `<span>` elements used in conjunction with HTML links whose only purpose is to apply a CSS class to the link. Instead, the CSS class attribute is applied directly to the link's `<a href="...">` element.
>
> There are three CSS styles for links previously used with embedded or embedding `<span>` elements:
>
> - `type-name-link`, which was applied to a `<span>` element inside the link. This was generated by passing `true` as argument for the `strong` parameter in various get*Link methods.
> - `member-name-link` and `search-tag-link`, which were applied to a `<span>` element wrapping the link.
>
> I could find no visual changes in the generated documentation with one exception:
>
> [Class and interface links in the second column of "Uses" page tables][1] were previously wrapped in a `<span class="member-name-link">` element causing the whole class name (including non-linked parts such as type parameters) to be displayed with a bold font. With this change, the `type-name-link` style is applied only to the active link, leaving non-linked parts such as type parameters with normal font-weight.
>
> [1]: https://download.java.net/java/early_access/jdk17/docs/api/java.base/java/util/class-use/Map.html#java.util
Hannes Wallnöfer has updated the pull request incrementally with one additional commit since the last revision:
JDK-8261499: Updates on feedback from code review
- Updated comments in markup.Links
- Rename LinkInfoImpl to HtmlLinkInfo and LinkFactoryImpl to HtmlLinkFactory
- Add HtmlDocletWriter.getDocLink(context, typeElement, element, label) overload
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/2516/files
- new: https://git.openjdk.java.net/jdk/pull/2516/files/875fb7da..e9f1c29f
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2516&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2516&range=00-01
Stats: 203 lines in 24 files changed: 16 ins; 7 del; 180 mod
Patch: https://git.openjdk.java.net/jdk/pull/2516.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/2516/head:pull/2516
PR: https://git.openjdk.java.net/jdk/pull/2516
More information about the javadoc-dev
mailing list