<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Add key-driven smooth scrolling to macOS"
   href="https://bugs.webkit.org/show_bug.cgi?id=228009#c9">Comment # 9</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Add key-driven smooth scrolling to macOS"
   href="https://bugs.webkit.org/show_bug.cgi?id=228009">bug 228009</a>
              from <span class="vcard"><a class="email" href="mailto:thorton@apple.com" title="Tim Horton <thorton@apple.com>"> <span class="fn">Tim Horton</span></a>
</span></b>
        <pre>Comment on <span class=""><a href="attachment.cgi?id=433969&action=diff" name="attach_433969" title="Patch">attachment 433969</a> <a href="attachment.cgi?id=433969&action=edit" title="Patch">[details]</a></span>
Patch

View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=433969&action=review">https://bugs.webkit.org/attachment.cgi?id=433969&action=review</a>

<span class="quote">>> Source/WebCore/page/EventHandler.cpp:4323
>> +    return view->scrollAnimator().beginKeyboardScrollAnimation(scroll, event);

> Are we always beginning here, or might the animation already be running from a previous event?</span >

It's the beginning /of a scroll/ (or a scroll gesture, or whatever you want to call it), but you're right, the animation might already be running.

<span class="quote">>> Source/WebCore/platform/ScrollAnimator.cpp:316
>> +            axesToApplySpring = perpendicularAbsoluteUnitVector(direction);

> Why is anything happening on the perpendicular axis? Wouldn't a keyboard scroll just be on one axis?</span >

(It took me a while to re-figure this out: while the key is down, the spring is not engaged in the axis of scrolling motion (instead, the force comes from the key), but we still want to let previous animations that might have stretched the string perpendicular to the current animation keep running to completion.)</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>