On Wed, 21 Oct 2026 21:10:53 GMT, Marius Hanl <[email protected]> wrote:
>> modules/javafx.controls/src/main/java/javafx/scene/control/skin/VirtualFlow.java
>> line 2874:
>>
>>> 2872:
>>> 2873: if (removedCells != null) {
>>> 2874: sheetChildren.removeAll(removedCells);
>>
>> we still have a regression: a freshly released cell with index=-1 can be
>> reused by a listener (if a listener scrolls, for example).
>
> but we are removing before, we are removing now. How can this regress? If a
> listener scrolls, layout should be called again because there is much more to
> do.
- a cell has its index set to -1
- the listener scrolls **synchronously**, grabbing a cell with index=-1, sets
new index
- the now active cell is removed
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2308#discussion_r4342033122
