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

Tejpal Rebari trebari at openjdk.java.net
Wed Mar 31 05:22:20 UTC 2021


On Wed, 24 Mar 2021 08:27:40 GMT, Prasanta Sadhukhan <psadhukhan 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
>
> I am not sure if this is the correct fix, as then we would have to remove setOpaque call for JTree, JToolTip, JRootPane etc where we might face same installProperty issue. 
> Also, if we have to change this, we then need to change "autoScrolls" property too which is set in this 2 component and setUIProperty() can change those too.
> Probably, we need to check if the value to be set is different in setUIProperty and then set it 
> or 
> override setUIProperty per component basis (as it is done for JTable) and then not check for OPAQUE_SET flag for opaque property accordingly.

@prsadhuk  @mrserb is there any more comments on this.

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

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


More information about the swing-dev mailing list