<table><tr><td style="">sander added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D20437">View Revision</a></tr></table><br /><div><div><p>How did you determine the value of the <tt style="background: #ebebeb; font-size: 13px;">damping</tt> variable?  You write</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">With this patch this automatic scrolling goes at 60 fps instead of 10, I made it so that the speed of the scroll is the same.</pre></div>

<p>If somebody in the future changes the scroll speed, will you have to update your patch?  Wouldn't it be better to have only one constant that determines the speed of both regular scrolling and automatic scrolling?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20437">https://phabricator.kde.org/D20437</a></div></div><br /><div><strong>To: </strong>kezik, Okular, aacid<br /><strong>Cc: </strong>sander, ngraham, aacid, ahmedbilal, okular-devel, kezik, joaonetto, tfella, darcyshen<br /></div>