RFR: 8323511: Scrollbar Click jumps inconsistent amount of pixels [v3]
eduardsdv
duke at openjdk.org
Mon May 6 16:56:59 UTC 2024
On Mon, 6 May 2024 16:34:22 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision:
>>
>> JDK-8323511: Fix scrolling when clicking the scrollbar track when only one cell is visible
>
> modules/javafx.controls/src/main/java/javafx/scene/control/skin/VirtualFlow.java line 1804:
>
>> 1802:
>> 1803: /**
>> 1804: * The amount of pixels by which to adjust the position when the the scroll bar track is clicked.
>
> should it be the height of the viewport, so to speak?
Yes, that would also be my favorite choice for the click on the scrollbar track as well as for the PageUp/PageDown.
But for reasons of backwards compatibility, I didn't want to change it here.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1326#discussion_r1591295131
More information about the openjfx-dev
mailing list