RFR CODETOOLS-7902339: Handle different look and feels in JFileChooserOperator
abdul.kolarkunnu at oracle.com
Tue Oct 30 04:56:29 UTC 2018
Changeset : http://cr.openjdk.java.net/~akolarkunnu/7902339/filechooser.txt
From: Alexandre (Shura) Iline
Sent: Monday, October 29, 2018 9:56 PM
To: Muneer Kolarkunnu <abdul.kolarkunnu at oracle.com>
Cc: jemmy-dev at openjdk.java.net
Subject: Re: RFR CODETOOLS-7902339: Handle different look and feels in JFileChooserOperator
Do you want me to push it for you?
> On Oct 29, 2018, at 1:49 AM, Muneer Kolarkunnu <abdul.kolarkunnu at oracle.com> wrote:
> Hi Shura,
> Thanks for review, please see inline comments, updated webrev: http://cr.openjdk.java.net/~akolarkunnu/7902339/webrev.01/
> -----Original Message-----
> From: Alexandre (Shura) Iline
> Sent: Thursday, October 25, 2018 11:48 PM
> To: Muneer Kolarkunnu <abdul.kolarkunnu at oracle.com>
> Cc: jemmy-dev at openjdk.java.net
> Subject: Re: RFR CODETOOLS-7902339: Handle different look and feels in JFileChooserOperator
> The JFileChooserOperator.java changes look great.
> 1. You can use @BeforeMethod even with data provider, it seems. There a seem to be a way to pass parameters to @BeforeMethod. Have you tried that?
>>>> Done it, it really reduced lots of code duplication, done similar changes in JInternalFrameOperatorTest.java and JInternalFrameOperatorCloseTest.java tests also.
> 2. I am worried about hardcoding “Desktop” even in the tests. That would mean that Jemmy tests only works on English windows. Should we think of localization resource bundles for this case?
>>>> Changed this to use API FileSystemView.getFileSystemView().getHomeDirectory().
> 3. Please bump minor version
>> On Oct 23, 2018, at 11:18 PM, Muneer Kolarkunnu <abdul.kolarkunnu at oracle.com> wrote:
>> Hi All,
>> Please review fix for the enhancement: https://bugs.openjdk.java.net/browse/CODETOOLS-7902339
>> Webrev : http://cr.openjdk.java.net/~akolarkunnu/7902339/webrev.00/
>> There are multiple issues in different L&Fs.
>> In GTK and Motif L&F:
>> There are two JLists, one is to list folders and second one one is to list files, but current JFileChooserOperator implementation didn't consider this, it expects only one JList that is to list files.
>> Windows & Windows Classic L&F:
>> There is no 'Go Home' button, but there is a toggle button to go desktop. In Windows platform 'Go Home' button usually(In metal and nimbus L&F) navigates to Desktop only.
More information about the jemmy-dev