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

Tejpal Rebari trebari at openjdk.java.net
Thu Apr 1 14:46:35 UTC 2021


On Wed, 31 Mar 2021 20:34:51 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

>> Tejpal Rebari has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Added default value check for opaque property
>
> Please wait for the @prsadhuk approval as well.

I have fixed the issue with JTree, JTooltip and JViewport same way.
While fixing for JTooltip discovered that default value for JToolTip opaque property for Nimbus LookAndFeel is set to false in the skin.laf file. This value was never taken into account because the setOpaque(true) from the JTooltip was overriding this.
As we are now removing it from the constructor of JTooptip, NimbusLAF is setting to the false.
To keep the behaviour same as before i have set it to true for NimbusLAF.

Also modified the test to check for JTooltip , JTree and JViewport.

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

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


More information about the swing-dev mailing list