On Thu, 17 Oct 2024 18:53:53 GMT, Phil Race <p...@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.
>
> "+ "Using mouse drag the header of the last culumn so the last column becomes 
> the first one.""
> 
> culumn -> column 
> 
> And would it be better to use PassFailJFrame for this test ?

@prrace Yes, it is better to use PassFailJFrame for this test and I have 
created a bug for the conversion https://bugs.openjdk.org/browse/JDK-8342663. I 
will work on this. I'm not mixing it with this fix as converting this test 
alone to use PassFailJFrame may not be feasible as all the tests inside the 
folder java/awt/a11y has a common abstract parent class 
AccessibleComponentTest, so it will be better to do the conversion for the 
whole set of tests inside java/awt/a11y folder together.

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

PR Comment: https://git.openjdk.org/jdk/pull/21549#issuecomment-2425724751

Reply via email to