On Tue, 20 Feb 2024 14:40:05 GMT, Hannes Wallnöfer <hann...@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
>
> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/resources/stylesheet.css
>  line 730:
> 
>> 728: }
>> 729: .col-first, .col-first {
>> 730:     font-size:0.93em;
> 
> Is this actually necessary, and if so, where exactly? It seems that the 
> elements affected by this change already have the following applied, which 
> should have the same effect:
> 
>     .summary-table > div {
>         overflow: auto hidden;
>     }

API doc(.html) generated using this stylesheet.css didn't had overflow: auto 
applied from .summary-table > div element but overflow been applied from 
.col-first, .col-second, .col-constructor-name which caused our achecker 
failure. .summary-table doesn't have affect.

Please find the below stylesheet.css used in one of our javadoc generated 
failed API doc

.summary-table > div, .details-table > div {
    text-align:left;
    padding: 8px 3px 3px 7px;
}
.col-first, .col-second, .col-last, .col-constructor-name, 
.col-summary-item-name {
    vertical-align:top;
    padding-right:0;
    padding-top:8px;
    padding-bottom:3px;
}
.table-header {
    background:#dee3e9;
    font-weight: bold;
}
.col-first, .col-first {
    font-size:13px;
}
.col-second, .col-second, .col-last, .col-constructor-name, 
.col-summary-item-name, .col-last {
    font-size:13px;
}
.col-first, .col-second, .col-constructor-name {
    vertical-align:top;
    overflow: auto;
}

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

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

Reply via email to