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