RFR: 8360462: [macosx] row selection not working with Ctrl+Shift+Down/Up in AquaL&F [v3]

Prasanta Sadhukhan psadhukhan at openjdk.org
Thu Jun 26 05:16:20 UTC 2025


On Thu, 26 Jun 2025 04:55:01 GMT, Abhishek Kumar <abhiscxk at openjdk.org> wrote:

>> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix
>
> test/jdk/javax/swing/JTree/TestTreeRowSelection.java line 62:
> 
>> 60:                 frame.pack();
>> 61:                 frame.setLocationRelativeTo(null);
>> 62:                 frame.setVisible(true);
> 
> You may move UI implementation code to helper method.

Not needed strictly..good to have linear flow..

> test/jdk/javax/swing/JTree/TestTreeRowSelection.java line 67:
> 
>> 65:             robot.waitForIdle();
>> 66:             robot.delay(1000);
>> 67:             int selectedRowCount = tree.getSelectionCount();
> 
> `tree.getSelectionCount()` on EDT ?

ok

> test/jdk/javax/swing/JTree/TestTreeRowSelection.java line 82:
> 
>> 80:             System.out.println("rows selected " + curSelectedRowCount);
>> 81:             if (curSelectedRowCount != selectedRowCount + 2) {
>> 82:                 throw new RuntimeException("ctrl+shift+down not working");
> 
> Does it sound better ?
> Suggestion:
> 
>                 throw new RuntimeException("ctrl+shift+down does not select next row");

ok

> test/jdk/javax/swing/JTree/TestTreeRowSelection.java line 89:
> 
>> 87:             robot.keyRelease(KeyEvent.VK_UP);
>> 88:             robot.keyRelease(KeyEvent.VK_SHIFT);
>> 89:             robot.keyRelease(KeyEvent.VK_CONTROL);
> 
> Helper method can be used to perform robot operations.

not needed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25966#discussion_r2168122496
PR Review Comment: https://git.openjdk.org/jdk/pull/25966#discussion_r2168122791
PR Review Comment: https://git.openjdk.org/jdk/pull/25966#discussion_r2168122589
PR Review Comment: https://git.openjdk.org/jdk/pull/25966#discussion_r2168122892


More information about the client-libs-dev mailing list