RFR: 8359599: Calling refresh() for all virtualized controls recreates all cells instead of refreshing the cells [v6]

Marius Hanl mhanl at openjdk.org
Tue Nov 4 21:53:37 UTC 2025


On Tue, 4 Nov 2025 18:38:12 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Marius Hanl has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   add copyright
>
> 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

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1830#discussion_r2492141465


More information about the openjfx-dev mailing list