RFR: 8359599: Calling refresh() for all virtualized controls recreates all cells instead of refreshing the cells [v6]
Kevin Rushforth
kcr at openjdk.org
Wed Nov 5 19:13:59 UTC 2025
On Tue, 4 Nov 2025 21:50:38 GMT, Marius Hanl <mhanl at openjdk.org> wrote:
>> tests/performance/control/src/table/TableRefreshTest.java line 77:
>>
>>> 75: refreshBtn.setOnAction(_ -> tv.refresh());
>>> 76: Button recreateBtn = new Button("Recreate");
>>> 77: recreateBtn.setOnAction(_ -> tv.getProperties().put("recreateKey", true));
>>
>> Question: Can this be done with public API? It's OK as is, since it is a test, but I figured I'd ask.
>
> Note like this, but setting a new `TableRow` factory will essentially trigger and do the same:
> https://github.com/openjdk/jfx/blob/a4e5451373beed93942e0bf2b6c8b81d3e6ad66d/modules/javafx.controls/src/main/java/javafx/scene/control/skin/TableViewSkinBase.java#L331-L337
OK, thanks. I would leave the test as you have it.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1830#discussion_r2495787816
More information about the openjfx-dev
mailing list