On Wed, 6 Mar 2024 09:25:51 GMT, psoujany <d...@openjdk.org> wrote:

>> 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?
>
> @hns Could you please validate my comment. Thank you.

After having spent spent some time researching the problem and the proposed 
solution, I find that the premise of this issue is wrong. With current versions 
of supported browsers on my computer (Firefox, Chrome, Safari) the scrollable 
tabs can be focused using the tab key (in Safari the "press tab to highlight 
items" option has to be enabled in the advanced settings). It may be that this 
is a new addition in Chrome and Safari, and that they did not support this at 
the time when the issue was detected and filed.

Apart from that, I don't think that the proposed change would be a good fix for 
the problem. By removing the `overflow: auto` declarations in the given 
`<div>s`, overflow becomes invisible, or overflows the layout in the case of 
the rightmost column. Making content inaccessible for all users is not a good 
solution for the problem of content being inaccessible to keyboard users, and 
breaking the layout of the page is not desirable either.

Given that mainstream browsers seem to have followed Firefox and converted on 
making scrollable elements tabbable I would not consider this a problem in our 
documentation.

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

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

Reply via email to