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