RFR: 8356770: TreeTableView not updated after removing a TreeItem with children and adding it to another parent [v2]
Andy Goryachev
angorya at openjdk.org
Fri Nov 14 17:31:04 UTC 2025
On Fri, 14 Nov 2025 09:30:29 GMT, Marius Hanl <mhanl at openjdk.org> wrote:
>> Ziad El Midaoui has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Add test
>
> modules/javafx.controls/src/test/java/test/javafx/scene/control/TreeTableViewTest.java line 6501:
>
>> 6499: }
>> 6500:
>> 6501: private static double disclosureX(TreeTableRow<String> row) {
>
> I know IntelliJ makes methods static by default (when extracting), but I don't think it makes sense here
I think it's perfectly fine to have a static method here (and in tests in general).
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1971#discussion_r2528340663
More information about the openjfx-dev
mailing list