<Swing Dev> [13] Review Request: 8225146 Accessibility issues in javax/swing/plaf/nimbus/doc-files/properties.html

[Sergey Bylokhov]

On 12/06/2019 07:09, Alexey Ivanov wrote:
> However, I'd prefer a more “verbose” version of <style> element, I mean with added line-breaks and indents unless we're saving disk space.
> I also wonder whether it's preserved intact after passing through javadoc.

I tried to save the space before the copyright header, and not to move it to the middle of the page.

> I'd rather leave bold rendering for keys, I think it makes sense. Yet it's up to you to decide.

Something like this?
http://cr.openjdk.java.net/~serb/8225146/webrev.03
http://cr.openjdk.java.net/~serb/8225146/webrev.03/raw_files/new/src/java.desktop/share/classes/javax/swing/plaf/nimbus/doc-files/properties.html

> 
> The <caption> element is optional. Does it make any difference to have one if it's hidden? With “display: none” it will not be visible to assistive technologies such as screen reader, will it?

The "caption" tag is used across our javadocs because in most cases the content of the "caption" tag is a content of the "summary" tag which was deprecated in html5, I guess at some point we will need to migrate to "aria-label" or something like this.

> 
> 
> I'm still for lower-case “and” in the page title and <h1>. But it's mostly nitpicking…
> 
>>> I tried to set the style to tbody itself and it does not work, will check your suggestion below.
>>>
>>> But before that, I would like to confirm that the "bold-version"(webrev.00) is not good enough?
>>>
>>>> (The previous version defaulted to bold rendering.)
>>>> If yes, is it possible to add a stylesheet to handle this for the entire page?
>>>> <style type="text/css">
>>>>      tbody th {
>>>>          font-weight: normal;
>>>>      }
>>>> </style>
>>>>
>>>>
>>>> Should the title of the page use title capitalization?
>>>> In this case, “Used” should be capitalized; at the same time “and” should be lower-case. Thus the title would be “Colors Used in Nimbus Look and Feel”.
>>>>
>>>> “How to Set the Look and Feel” [1] article in the Java Tutorials follows this capitalization style.
>>>>
>>>> Regards,
>>>> Alexey
>>>>
>>>> [1] https://docs.oracle.com/javase/tutorial/uiswing/lookandfeel/plaf.html
>>>>
>>>> On 11/06/2019 12:41, Sergey Bylokhov wrote:
>>>>> Hello.
>>>>> Please review the fix for JDK 13.
>>>>>
>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8225146
>>>>> Fix: http://cr.openjdk.java.net/~serb/8225146/webrev.01
>>>>>
>>>>> <SNIP>


-- 
Best regards, Sergey.