RFR: 8341453: java/awt/a11y/AccessibleJTableTest.java fails in some cases where the test tables are not visible [v3]
Abhishek Kumar
abhiscxk at openjdk.org
Mon Oct 21 15:30:26 UTC 2024
On Mon, 21 Oct 2024 10:19:37 GMT, Manukumar V S <mvs at openjdk.org> wrote:
>> This issue can be fixed by setting the preferred scrollable viewport size of the two tables using
>> JTable table = new JTable(data, columnNames);
>> + table.setPreferredScrollableViewportSize(table.getPreferredSize());
>> JTable secondTable = new JTable(data, columnNames);
>> + secondTable.setPreferredScrollableViewportSize(secondTable.getPreferredSize());
>>
>> The other changes in imports section are just re-arrangements and expansion of imports.
>>
>> Testing: This is a manual test and it is tested and verified manually in my local system
>> Screenshots of before and after the fix are attached in the bug.
>
> Manukumar V S has updated the pull request incrementally with one additional commit since the last revision:
>
> Review comments fixed : Copyright year added, re-arranged imports, removed author tag
test/jdk/java/awt/a11y/AccessibleJTableTest.java line 32:
> 30: * @requires (os.family == "windows" | os.family == "mac")
> 31: */
> 32:
You may remove extra blank line probably.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21549#discussion_r1809041506
More information about the client-libs-dev
mailing list