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