<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK] Add kinetic scrolling"
href="https://bugs.webkit.org/show_bug.cgi?id=155750#c22">Comment # 22</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK] Add kinetic scrolling"
href="https://bugs.webkit.org/show_bug.cgi?id=155750">bug 155750</a>
from <span class="vcard"><a class="email" href="mailto:cgarcia@igalia.com" title="Carlos Garcia Campos <cgarcia@igalia.com>"> <span class="fn">Carlos Garcia Campos</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=155750#c20">comment #20</a>)
<span class="quote">> Created <span class=""><a href="attachment.cgi?id=281711&action=diff" name="attach_281711" title="Patch - new features + bugfix">attachment 281711</a> <a href="attachment.cgi?id=281711&action=edit" title="Patch - new features + bugfix">[details]</a></span>
> Patch - new features + bugfix
>
> I've used this patch in an internal project and made some modifications that
> you may find useful. I am posting the updated patch without ChangeLogs, feel
> free to update it or let me know if you prefer if I do it.
>
> The changes included in this patch (on top of latest one from Adrien):
> - overshoot animations support (ScrollAnimationKinetic.cpp)
> - bugfix for a crash when starting a drag while scrolling
> (GestureController.cpp)
> - build fix for gtk < 3.20 (PlatformWheelEventGtk.cpp)
> - bugfix for gtk < 3.20 to properly copy NativeWebWheelEvent (is_stop is not
> available in gtk < 3.20 and the phase/momentumPhase were not being properly
> copied on the NativeWebWheelEvent copy constructor, making the page "jump"
> during scroll instead of starting momentum scroll -
> NativeWebWheelEventGtk.cpp)
> - respect scrollbar policy (ScrollAnimationKinetic.cpp +
> ScrollAnimatorGtk.cpp)
>
> That should be it, please let me know what you think.</span >
Thanks! I think Adrien should include crashes and build fixes in his patch when submitting a new version and you should open a new bug depending on this one to add the overshooting support on top of Adrien's patch.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>