<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a comment.<br />This revision now requires changes to proceed.
</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/D9857" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Finally got a chance to give this a whirl.</p>

<p>First, let's make sure we're talking about the same thing, because scroll terminology can otherwise be a bit imprecise:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item"><strong>Smooth scrolling:</strong> Scrolling using the scroll wheel, arrow keys, PageUp/PageDown keys, or the spacebar will results in an animated transition effect rather than the view "jumping" to the new location</li>
<li class="remarkup-list-item"><strong>Pixel scrolling:</strong> Doing a two-finger scroll gesture on a touchpad scrolls scroll pixel-by pixel proportionally with your fingers' movement, rather than the movement being interpreted as a scroll wheel and advancing the view by a fixed or user-configurable number of lines</li>
</ul>

<p>Okular already does pixel scrolling, but does <strong>not</strong> do smooth scrolling. However, your patch also does not implement smooth scrolling; it allows the user to override the default number of "lines" per scroll wheel tick or arrow key press, and instead scroll by a user-configurable number of pixels.</p>

<p>Without tweaking the "Scroll Step" parameter, scrolling using the arrow keys, scroll wheel, or touchpad becomes interminably slow. Increasing Scroll Step gradually  returns the behavior closer to the default.</p>

<p>As currently implemented, I'm not sure what value this adds to the typical Okular user. I played with the new setting a bunch but all of the values felt worse than the default. Can you maybe describe your use case, and what value this brings to you?</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/D9857" rel="noreferrer">https://phabricator.kde.org/D9857</a></div></div><br /><div><strong>To: </strong>rmacmaster, okular-devel, Okular, ngraham<br /><strong>Cc: </strong>ngraham, Okular, michaelweghorn, simgunz, aacid<br /></div>