On Tue, 3 Jan 2023 23:56:12 GMT, Nir Lisker <nlis...@openjdk.org> wrote:

>> A simple fix for the graphics of a `CheckBoxTreeCell` not showing after 
>> expand/collapse events. The graphics node needs to be released from the 
>> previous checkbox so that it doesn't serve as the graphics of more than one 
>> checkbox at once. If it does, only one checkbox (its skin) can actually use 
>> it as a scenegraph child, and it's not always the correct one.
>> 
>> Added also a manual test app as I'm unsure about how to do an automatic test 
>> for this. The test app will be useful for later fixes, so I suggest adding 
>> it anyway. To test the fix, simply expand and collapse the root note 
>> repeatedly while it has a child. Before the fix the graphic will disappear 
>> after a few tries, after the fix it won't. You can test other hierarchies as 
>> well by adding children to the tree.
>
> Nir Lisker has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Removed thrown exception

Thanks for the quick review!

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

PR: https://git.openjdk.org/jfx/pull/970

Reply via email to