RFR: 8321722: Tab header flickering when dragging slowly other tabs and reordering uncompleted
Andy Goryachev
angorya at openjdk.org
Mon Dec 11 19:04:34 UTC 2023
On Mon, 11 Dec 2023 18:51:09 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
>> This PR prevents changes in direction when dragging a tab header over a TabPane control, if delta value is zero, which can happen with slow drag events, in order to prevent starting and stopping too frequently the animation of the target tab header in opposite directions.
>
> modules/javafx.controls/src/main/java/javafx/scene/control/skin/TabPaneSkin.java line 2162:
>
>> 2160: }
>> 2161: // Stop dropHeaderAnim if direction of drag is changed
>> 2162: if (dragDirection != 0 && prevDragDirection != dragDirection) {
>
> Did you mean `dragDelta != 0`?
it is sort of equivalent in this case...
I think this code is correct.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1304#discussion_r1423002773
More information about the openjfx-dev
mailing list