On Mon, 29 Nov 2021 11:56:45 GMT, Johan Vos <j...@openjdk.org> wrote:
> After (re)setting the number of elements, make sure to do at least some > estimation of the total size. > Added a testcase for this scenario. modules/javafx.controls/src/main/java/javafx/scene/control/skin/VirtualFlow.java line 903: > 901: recalculateAndImproveEstimatedSize(2); > 902: > 903: cellCount.set(value); Can this be implemented in a way that doesn't violate the property contract? Since this property is public API, `setCellCount(int)` should just call `cellCountProperty().set(int)`. Maybe you can extract the composite operation here into a private method and use that instead of `setCellCount(int)`. `position` has the same problem. ------------- PR: https://git.openjdk.java.net/jfx/pull/683
