On Wed, 21 Feb 2024 14:41:29 GMT, Hannes Wallnöfer <hann...@openjdk.org> wrote:
>> 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;
>> }
>
> The current stylesheet has the following rule at line 688:
>
>
> .summary-table > div, .details-table > div {
> text-align:left;
> padding: 8px 3px 3px 7px;
> overflow: auto hidden;
> scrollbar-width: thin;
> }
>
>
> Granted, the `overflow` declaration was added relatively recently (October or
> last year). Does the problem still persist with this?
As you mentioned earlier, overflow:auto at line 730 is getting overlapped by
line 688. Still we see achecker failing with `element_scrollable_tabbable`
violation. As summary-table is a grid, I think overflow may not have impact.
Please correct me. Thank you.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17819#discussion_r1499025866
