RFR: 8325690: The scrollable element <div> with non-interactive content is not tabbable [v23]

[Hannes Wallnöfer]

On Mon, 27 May 2024 09:50:38 GMT, psoujany <duke at openjdk.org> wrote:

>> The scrollable element `<div>` with non-interactive content is not tabbable. Grid columns in the javadoc stylesheet has overflow: auto, which is failing Accessibility checks.
>> https://bugs.openjdk.org/browse/JDK-8325690
>
> psoujany has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Add tab order to plain text
>  - Add tab order to plain text

src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/Table.java line 349:

> 347:              * For more information on the `widget_tabbable_single` rule, please refer to the following documentation:
> 348:              * <a href="https://able.ibm.com/rules/archives/latest/doc/en-US/widget_tabbable_single.html#%7B%22message%22%3A%22Component%20with%20%5C%22tablist%5C%22%20role%20has%20more%20than%20one%20tabbable%20element%22%2C%22snippet%22%3A%22%3Cdiv%20tabindex%3D%5C%220%5C%22%20role%3D%5C%22tablist%5C%22%20class%3D%5C%22col-second%20odd-row-color%20method-summary-table%20method-summary-table-tab2%20method-summary-table-tab3%5C%22%3E%22%2C%22value%22%3A%5B%22VIOLATION%22%2C%22POTENTIAL%22%5D%2C%22reasonId%22%3A%22fail_multiple_tabbable%22%2C%22ruleId%22%3A%22widget_tabbable_single%22%7D">widget_tabbable_single</a>
> 349:              */

I think this new comment is a bit excessive. Can it be boiled down to a single sentence, maybe one or two lines? Also, there's no need to use `/**` doc comment style here. For example (but feel free to improve):

    // Only make element tabbable if it doesn't contain tabbable content.

Regarding the link, I don't think the content after the `#` is necessary. I would prefer the following form:

    // https://able.ibm.com/rules/archives/latest/doc/en-US/widget_tabbable_single.html

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17819#discussion_r1622499481