RFR: 8299522: JFilechooser open button size is incorrectly shown

Alexey Ivanov aivanov at openjdk.org
Tue Jan 10 13:15:53 UTC 2023


On Tue, 10 Jan 2023 04:50:08 GMT, Tejesh R <tr at openjdk.org> wrote:

> > Taking the above into account, [JDK-8299522](https://bugs.openjdk.org/browse/JDK-8299522) is **Not an Issue**.
> 
> @aivanov-jdk Agree to your point. But I added the default text and tooltip as a fallback text which is already in aqua L&F. It's always safe to add a default text when user doesn't add explicitly, since it affects the button size in some L&F (like Metal) and empty text button in some other L&F (like windows).

Yet it is explicitly stated in the documentation that you are expected to explicitly provide the text for the Approve button if you use `CUSTOM_DIALOG` type.

This case is handled gracefully when the `showDialog` method is used, the Approve button has `Open` text at least in Windows and Metal L&Fs. This happens in Aqua L&F too.

Thus, this situation is already handled by the API which is most commonly used with `JFileChooser`.

I suggest handling the missing text for the Approve button if a custom dialog is used with adding a `JFileChooser` instance as component by falling back to the `Open` type.

If you prefer going this route and add new properties, I suggest following what's used in Aqua L&F that is `choose-` prefix for the properties and member names.

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

PR: https://git.openjdk.org/jdk/pull/11901



More information about the client-libs-dev mailing list