[Webkit-unassigned] [Bug 224182] [GTK] REGRESSION: Kinetic scrolling on touchpad doesn't work with async scrolling off

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=224182

--- Comment #2 from Chris Lord <clord at igalia.com> ---
Drive-by comments;

It does seem odd that we would stop the animation but not clear the scroll history - I'm reasonably sure this worked as expected when I originally checked in momentum scrolling, I'd be concerned that this might cause some unexpected behaviour (though obviously there currently is unexpected behaviour, so that's a bit of a moot point).

If all of these things work with async and non-async scrolling, with GTK and WPE, I think this should be fine:
- Touchscreen or touchpad scrolling without triggering a kinetic scroll animation
- Touchscreen or touchpad scrolling and triggering a kinetic scroll animation
- Increasing the velocity of a kinetic scroll animation with touchscreen or touchpad scrolling (momentum scrolling)
- Stopping a kinetic scroll animation with touchscreen or touchpad scrolling
- Smooth scrolling with a mouse-wheel and keyboard
- Mouse-wheel or keyboard scrolling interrupting animated scrolling

It's a lot to test... We briefly chatted on IRC, to reiterate what we discussed there, I think it'd be good to get API tests written for both the animation classes and the event-handling so that we can establish expected sequences and behaviours.

Given this is currently completely untested, I don't think that should hold up this particular patch, but it'd establish a good precedent to at least add a test that would catch this particular failure (unexpected clearing of scroll history when calling scroll-to api).

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20210405/8f1e2f1e/attachment-0001.htm>