RFR: 8359599: Calling refresh() for all virtualized controls recreates all cells instead of refreshing the cells [v2]
Marius Hanl
mhanl at openjdk.org
Fri Oct 3 14:52:02 UTC 2025
On Fri, 3 Oct 2025 05:25:39 GMT, John Hendrikx <jhendrikx at openjdk.org> wrote:
> I think the changes look good. I'm a bit confused in the performance table with what is meant with the `50 ms -> 0 ms` in the "after" cases though?
Every `refresh()` will trigger 2 layouts for some reason, where the second one does nothing as nothing is dirty, so basically a noop. I can have a look into that (maybe as a follow up?) but I remember that this happens sometimes in general for the `VirtualFlow` and we should check that generally at one point.
@andy-goryachev-oracle solved that problem in the `RichTextArea` by isolating the method which should be called e.g. two times (due to e.g. `ScrollBar`s) instead of a real relayout.
-------------
PR Comment: https://git.openjdk.org/jfx/pull/1830#issuecomment-3366012168
More information about the openjfx-dev
mailing list