RFR: 8334874: Horizontal scroll events from touch pads should scroll the TabPane tabs [v3]
Andy Goryachev
angorya at openjdk.org
Wed Jul 24 21:22:39 UTC 2024
On Tue, 25 Jun 2024 21:47:42 GMT, Jose Pereda <jpereda at openjdk.org> wrote:
>> This PR considers the horizontal scroll events that can be generated on a trackpad, on an horizontally sided `TabPane` control (top or bottom), adding to the existing vertical scroll events from mouse wheel and trackpad, to scroll its tabs.
>>
>> Therefore, scrolling a `TabPane` will behave in the same way that `ScrollBar` does regarding those same events and control orientation (vertical/horizontal).
>
> Jose Pereda has updated the pull request incrementally with one additional commit since the last revision:
>
> Add tests for all four sides
I am ok with fixing the direction of vertical scrolling in a separate ticket, as well as RTL reversal of the horizontal scrolling, also as a separate ticket.
Left a couple of minor comments, otherwise looks good. I'll re-approve if you decide to make changes.
modules/javafx.controls/src/test/java/test/javafx/scene/control/TabPaneTest.java line 1345:
> 1343: }
> 1344: }
> 1345:
please remove unnecessary newline
-------------
Marked as reviewed by angorya (Reviewer).
PR Review: https://git.openjdk.org/jfx/pull/1486#pullrequestreview-2197767859
PR Review Comment: https://git.openjdk.org/jfx/pull/1486#discussion_r1690429539
More information about the openjfx-dev
mailing list