<Swing Dev> RFR: 8253266 : JList and JTable constructors should clear OPAQUE_SET before calling updateUI [v3]

Prasanta Sadhukhan psadhukhan at openjdk.java.net
Mon Apr 5 07:11:29 UTC 2021


On Mon, 5 Apr 2021 06:34:17 GMT, Tejpal Rebari <trebari at openjdk.org> wrote:

>> src/java.desktop/share/classes/javax/swing/plaf/nimbus/skin.laf line 27251:
>> 
>>> (failed to retrieve contents of file, check the PR for context)
>> In my opinion, instead of changing skin.laf we probably should update SynthToolTipUI.installDefaults() and add 
>> `LookAndFeel.installProperty(c, "opaque", Boolean.TRUE);`
>> 
>> similar to what we do for BasicToolTipUI.installDefaults. Will it not work?
>
> No, it doesn't work, i have checked.
> The change needs to be skin.laf file only.

I am not sure where you placed the code. I think it's need to be added after updateStyle() as updateStyle() uninstall the default and install for new style.
protected void installDefaults(JComponent c) {
        updateStyle(c);
    }

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

PR: https://git.openjdk.java.net/jdk/pull/3167


More information about the swing-dev mailing list