RFR: 8359599: Calling refresh() for all virtualized controls recreates all cells instead of refreshing the cells [v3]
Kevin Rushforth
kcr at openjdk.org
Thu Oct 23 16:22:03 UTC 2025
On Wed, 15 Oct 2025 19:07:13 GMT, Marius Hanl <mhanl at openjdk.org> wrote:
>> We should be careful not to over-specify this. The "in other words" part already suggests what it does. Maybe it could be made more explicit by adding "to match the underlying data source", like this?
>>
>>
>> * In other words, this forces the TableView to update what it is showing to
>> * the user to match the underlying data source.
>> * This is useful in cases where the underlying data source has
>> * changed in a way that is not observed by the TableView itself.
>
> I like this idea.
> If we want to explain what rebuild does, I would probably mention that this means the cells are updated. So maybe instead of `rebuild`, we could use something like `re-update`, maybe?
@Maran23 Can you update the javadoc for the four methods as Andy suggested above (possibly with my simplification) and then update the CSR Specification section to match? I think that's the only remaining thing needed to move this forward.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1830#discussion_r2455868260
More information about the openjfx-dev
mailing list