RFR: 8276553: ListView scrollTo() is broken after fix for JDK-8089589 [v3]

Kevin Rushforth kcr at openjdk.java.net
Fri Dec 3 18:50:14 UTC 2021


On Fri, 3 Dec 2021 11:25:43 GMT, Johan Vos <jvos at 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.
>
> Johan Vos has updated the pull request incrementally with one additional commit since the last revision:
> 
>   process reviewer comment (simplify call)

The fix and new test looks fine. I added one comment/question regarding the existing tests that check a count, but I'm OK with either answer.

Approved.

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

Marked as reviewed by kcr (Lead).

PR: https://git.openjdk.java.net/jfx/pull/683


More information about the openjfx-dev mailing list