On Wed, 21 Feb 2024 05:22:47 GMT, psoujany <d...@openjdk.org> wrote:
>> 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;
> }
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?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17819#discussion_r1497700367
