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