On Mon, 21 Oct 2024 10:19:37 GMT, Manukumar V S <m...@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

Reply via email to