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