On Wed, 10 Jan 2024 11:26:17 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>> Abhishek Kumar has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   ExpandedIconWrapper and CollapsedIconWrapper class merged together
>
> src/java.desktop/share/classes/javax/swing/plaf/synth/SynthTreeUI.java line 
> 82:
> 
>> 80:     private Icon expandedIconWrapper = new 
>> IconWrapper(IconType.EXPANDED);
>> 81: 
>> 82:     private Icon collapsedIconWrapper = new 
>> IconWrapper(IconType.COLLAPSED);
> 
> I guess you can pass in `collapsedIcon` or `expandedIcon`  to `IconWrapper 
> `class which can save it in `icon` field and
> Then IconWrapper class can just call `SynthGraphicsUtils.paintIcon(icon, 
> context, g, x, y, w, h);`

I tried with this approach first but images didn't render at all.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17294#discussion_r1447313431

Reply via email to