On Wed, 24 Mar 2021 08:27:40 GMT, Prasanta Sadhukhan <psadhuk...@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

Reply via email to