RFR: 8276553: ListView scrollTo() is broken after fix for JDK-8089589 [v2]
Johan Vos
jvos at openjdk.java.net
Tue Nov 30 11:02:42 UTC 2021
On Mon, 29 Nov 2021 17:00:10 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
>> Exactly. This ensures consistent behavior regardless of how a property value is set.
>
> I should add that this looks like a preexisting problem, but one that would be good to fix if possible.
Thanks for catching. I moved the additional code into the invalidated block, at the appropriate places.
Note that this caused some changes in the flow at runtime, where the number of evaluations of `updateItem` changed. Those changes are of constant order (and not linear with the amount of cells), and I made some changes to the test as they depended on hard numbers.
-------------
PR: https://git.openjdk.java.net/jfx/pull/683
More information about the openjfx-dev
mailing list