RFR: 8299522: Incorrect size of Approve button in custom JFileChooser [v12]
Alexey Ivanov
aivanov at openjdk.org
Wed Jan 18 10:34:29 UTC 2023
On Wed, 18 Jan 2023 10:18:05 GMT, Tejesh R <tr at openjdk.org> wrote:
> > > The test fails for me on macOS:
> > > ```
> > > Caused by: java.lang.RuntimeException: Approve button tooptip is null in FileChooser!
> > > at CustomApproveButtonTest.<init>(CustomApproveButtonTest.java:78)
> > > at CustomApproveButtonTest$1.run(CustomApproveButtonTest.java:48)
> > > ```
> > >
> > > I merged master into your PR branch to get the fix for [JDK-8300084](https://bugs.openjdk.org/browse/JDK-8300084) and re-built JDK before running the test.
> >
> > In your local setup?
Yes, in my local setup. I'm pretty sure it'll fail on mach5 too.
> > The fallback tooltip for approve button is null for aqua, we have to skip checking tooltip for aqua?
Should it rather be set to a non-null value?
Personally, I think the tooltip for the Approve button isn't needed as its purpose is obvious. However, the tooltip is added in all the L&F, so in custom case there should also be the tooltip.
-------------
PR: https://git.openjdk.org/jdk/pull/11901
More information about the client-libs-dev
mailing list