RFR: 6753661: JFileChooser font not reset after Look & Feel change [v6]

Alexey Ivanov aivanov at openjdk.org
Fri Feb 10 16:11:49 UTC 2023


On Fri, 10 Feb 2023 16:00:56 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> Issue is observed that after changing the Look & Feel from Metal to Nimbus and back to Metal, the Nimbus font continues to be used by a JFileChooser.
>> This is because Synth  `installDefaults `methods set the font, but its inverse methods `uninstallDefaults `do not remove them.
>> Fix is made to reset the font if it is set by L&F.
>
> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Add both null check

test/jdk/javax/swing/JFileChooser/JFileChooserFontReset.java line 70:

> 68:                     System.out.println("current font " + curFont);
> 69:                     if ((curFont != null && !curFont.equals(origFont)) ||
> 70:                         (origFont != null && !origFont.equals(curFont))) {

The newly added condition is redundant because `origFont.equals(curFont) == curFont.equals(origFont)`. (It's possible `equals` returns different values but it violates its contract then.)

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

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



More information about the client-libs-dev mailing list