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

Marius Hanl mhanl at openjdk.org
Wed Oct 15 19:10:37 UTC 2025


On Wed, 15 Oct 2025 16:59:29 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> I still don't know what "rebuild" means in this context.
>
> 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?

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

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


More information about the openjfx-dev mailing list