On Thu, 5 Dec 2024 09:24:13 GMT, Marius Hanl <mh...@openjdk.org> wrote:

>> modules/javafx.controls/src/main/java/javafx/scene/control/skin/TreeTableRowSkin.java
>>  line 228:
>> 
>>> 226:         if (disclosureNodeDirty) {
>>> 227:             updateDisclosureNodeAndGraphic();
>>> 228:             disclosureNodeDirty = false;
>> 
>> shouldn't `disclosureNodeDirty = false;` be moved to 
>> `updateDisclosureNodeAndGraphic();` ?
>> here and in L257, L216
>
> Could be an idea. I will check. I just tried to not make to many changes, but 
> this might be an easy one.

Checked the code, that is a safe improvement, so will do.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1644#discussion_r1876896331

Reply via email to