RFR: 8299522: Incorrect size of Approve button in custom JFileChooser [v9]

Alexey Ivanov aivanov at openjdk.org
Mon Jan 16 14:32:13 UTC 2023


On Mon, 16 Jan 2023 14:20:30 GMT, Tejesh R <tr at openjdk.org> wrote:

>> FileChooser Open/Approve button size is shown incorrectly when no Approve button text is set in `CUSTOM_DIALOG` type. Reason being that no default Approve Button text is returned during Dialog Type Property change. Since `null` is returned as Button string the Button size is incorrectly shown. The fix here addresses the issue by adding a default Approve Button Text when no text is set explicitly in case of `CUSTOM_DIALOG` type.
>> Automated test is attached which has been tested with multiple test runs.
>
> Tejesh R has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Added Aqua L&F -  linked to #12008 fix

Marked as reviewed by aivanov (Reviewer).

test/jdk/javax/swing/JFileChooser/CustomApproveButtonTest.java line 38:

> 36:  * @key headful
> 37:  * @summary JFileChooser Test to check for FileChooser's Approve Button Text
> 38:  *          when CUSTOM_DIALOG type is set and check getDefaultButton() (AquaL&F)

Suggestion:

 * @summary Verifies JFileChooser's Approve button text is non-null
 *          when CUSTOM_DIALOG type is set

I'd remove reference to Aqua L&F, it's an unimportant detail in my opinion.

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

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



More information about the client-libs-dev mailing list