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

Marius Hanl mhanl at openjdk.org
Sun Oct 12 14:52:43 UTC 2025


On Fri, 10 Oct 2025 22:35:07 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Changing this is out of scope for this PR.
>
> Having said that, I've always disliked using user properties for this sort of thing. It feels like a bit of a hack, so it might be worth filing a follow-up to evaluate a different approach in general (not just for this one instance of the pattern). I don't see this as a high priority, though.

Adding to what @kevinrushforth said, this is used everywhere. All layout constraints are implemented by using `Properties`. Changing anything there can break the layout.

If we just talking about the properties calling a method in the skin, I dislike the current way as well, but have no other idea really. Maybe a good topic for the mailing list then.

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

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


More information about the openjfx-dev mailing list