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