Inertial scrolling was just implmented in okular, which is awesome. However, scrolling with the browse (hand) tool (by left-clicking the document and pulling it down or up) now sticks for a good number of pixels before it starts moving, which causes it to jump for the first part of the scroll and is jarring for the user.
This is most noticeable, by the way, the more slowly you drag scroll the document.
I recognized that when I was playing arround with my new trackpoint, but couldn’t verify the link to smooth scrolling.
I assume you recently updated Okular, and recognized that you got smooth scrolling and this bug the same time.
When smooth scrolling will be reimplemented, we should test this.
Yes, I recently updated okular knowing that smooth scrolling was implemented and noticed immediately after the sticky drag scroll, since I view a lot of pdfs daily and my preferred scroll method is using the browse tool (since it's smoother than using the mouse wheel).
This is a QScroller feature almost for sure.
It's one of the configuration parameters that Qt documentation explicitly says to keep to the "platform optimized" values.
You feel it's too severe?
(btw, try it in your phone, it does the same)
It's not noticeable on Android because the scroll doesn't jump. It feels too severe to me and doesn't even exist using touch input in okular, as far as I can tell.
Can confirm. There seems to be a small initial threshold before the grab actually moves the document, at which point it jumps. Either the threshold or the jump should be removed.
A possibly relevant merge request was started @ https://invent.kde.org/graphics/okular/-/merge_requests/272
Git commit a5be0149ec627fbc086e5b26cbbe7be13cde49b8 by Oliver Sander, on behalf of Kezi Olio.
Committed on 15/09/2020 at 12:11.
Pushed by ngraham into branch 'release/20.08'.
Restore mouse drag scrolling, so it starts immediately
QScroller uses a minimum drag distance before drag scrolling starts by default.
This sets the minimum distance to 0.0, to restore the original behavior of the Browse tool.
M +1 -0 ui/pageview.cpp
Thanks for the fix!