Summary: | Scrolling issues with PageDown/PageUp navigation | ||
---|---|---|---|
Product: | [Applications] okular | Reporter: | mepublic+kdebugs |
Component: | general | Assignee: | Okular developers <okular-devel> |
Status: | VERIFIED FIXED | ||
Severity: | normal | CC: | aacid, ae, dennis.lissov, harrisonts, j.stryker92, johnjohnlys, jwork123nl, kdudka, keziolio123, kishore96, luke-jr+kdebugs, nate, oliver.sander, quantera, raullaasner, terry, triffid.hunter |
Priority: | NOR | ||
Version: | 1.10.1 | ||
Target Milestone: | --- | ||
Platform: | Arch Linux | ||
OS: | Linux | ||
See Also: | https://bugs.kde.org/show_bug.cgi?id=420755 | ||
Latest Commit: | https://invent.kde.org/graphics/okular/-/commit/e637e2ed519cb2dfe523981b2d6dc34a9855716c | Version Fixed In: | 1.11.2 |
Sentry Crash Report: |
Description
mepublic+kdebugs
2020-05-25 13:22:40 UTC
The following fix worked for me: diff --git a/ui/pageview.cpp b/ui/pageview.cpp index 50f8073ec..9bdb778ec 100644 --- a/ui/pageview.cpp +++ b/ui/pageview.cpp @@ -5375,7 +5376,7 @@ void PageView::slotScrollUp( int nSteps ) d->scroller->scrollTo(d->scroller->finalPosition() + QPoint(0,-100*nSteps), 0); }else{ if(d->scroller->finalPosition().y() > verticalScrollBar()->minimum()) - d->scroller->scrollTo(d->scroller->finalPosition() + QPoint(0, -verticalScrollBar()->rect().height() )); + d->scroller->scrollTo(d->scroller->finalPosition() + QPoint(0, -verticalScrollBar()->rect().height() ), 0); } } else if ( d->document->currentPage() > 0 ) @@ -5403,7 +5404,7 @@ void PageView::slotScrollDown( int nSteps ) d->scroller->scrollTo(d->scroller->finalPosition() + QPoint(0,100*nSteps), 0); }else{ if(d->scroller->finalPosition().y() < verticalScrollBar()->maximum()) - d->scroller->scrollTo(d->scroller->finalPosition() + QPoint(0, verticalScrollBar()->rect().height() )); + d->scroller->scrollTo(d->scroller->finalPosition() + QPoint(0, verticalScrollBar()->rect().height() ), 0); } } else if ( (int)d->document->currentPage() < d->items.count() - 1 ) The second argument of scrollTo is the animation speed, which I've set to zero. Thanks. I will try it over the weekend I want to add that the smooth scrolling with these limitations really ruins the experience of reading through a PDF presentation in the "fit page" mode. This is the case where I expect instant transitions because presentations sometimes contain slides where you're expected to see the difference between the two slides. Also, point (a) about missing the second press completely breaks the ability to quickly go N pages back ("let's check the phrase that was 3 slides ago"). I totally agree. Please add a switch to disable this feature. I used to browse the PDF by pressing the page up/down quickly, but now I cannot do this. > with these limitations
It's not a limitation of the smooth scrolling itself, I'm incredibly busy in this period and wasn't able to provide a decent fix for the other bug, those are all easy fixes but I literally don't have the time, feel free to disable all the offending stuff, I'll come back next month, sorry.
*** Bug 422629 has been marked as a duplicate of this bug. *** > ("let's check the phrase that was 3 slides ago")
Note that Arrow Left/Right still does instant transitions.
A possibly relevant merge request was started @ https://invent.kde.org/graphics/okular/-/merge_requests/238 Another vote that the "smooth" scrolling animations on PgUp/PgDn are really awful. It's unexpected and disrupts my flow, especially when I need to jump around in a document. No other program on my desktop including Dolphin, KWrite, or Gwenview has this. Also worth noting that the pixel-wise scrolling using my touchpad is much slower in Okular than any other program, almost unusable. I wasn't able to find a relevant bug report but it seems related to the the custom scrolling behavior in Okular. +1 to be able to toggle off all smooth scroll-related effects. Here's a another merge request that fixes the issue: https://invent.kde.org/graphics/okular/-/merge_requests/242 It's been waiting for a few weeks now. Here's my own solution which makes kinetic scrolling optional: https://invent.kde.org/raullaasner/okular/-/commit/bb921dfbc64c691ef4575587a15a94234b197234 It's similar to the another ones so I didn't start another merge request. Just using it until the official fix comes out. Are you sure that Dolphin has no scroll animation? Mine has. Or did you just figure out how to disable it? I can’t say anything about the touchpad scroll speed. When I had a touchpad, Okular 19.04 had normal scroll speed with two-finger scrolling. Okular 20.08 seems to have same scroll speed as other applications with the trackpoint (psmouse driver). I think you can come to BoF on Wednesday, where we want to discuss scrolling. See https://community.kde.org/Akademy/2020 I'm running Debian Bullseye so my versions may be a little behind. I have Dolphin 20.04.3 and I thought there was no animation, though it may actually be there but much faster. I don't really mind it at that speed, though I would still prefer no animation. It is noticeable browsing the web in Konqueror 5.0.97. The touchpad scroll speed may have to do with the fact that I changed the libinput Coordinate Transformation Matrix to speed up the cursor. I'll do some more testing. Thanks for the invite to the session! I love to see a project that encourages collaboration, and I have to say I'm more satisfied overall with KDE than any other desktop since making the switch several months ago. Really an exemplar for FOSS project management. I'll see if I can make it. Just to remind everyone, there's already !194 that was addressing this bug two months ago, by reverting to the old behavior. I can understand why you wouldn't want that patch given the circumstances, but really, the fix is sound, I shouldn't be still hearing people complain about this bug *** Bug 425492 has been marked as a duplicate of this bug. *** Fixed with https://invent.kde.org/graphics/okular/-/commit/e637e2ed519cb2dfe523981b2d6dc34a9855716c in Okular 1.11.2 *** Bug 421822 has been marked as a duplicate of this bug. *** (In reply to Nate Graham from comment #15) > Fixed with > https://invent.kde.org/graphics/okular/-/commit/ > e637e2ed519cb2dfe523981b2d6dc34a9855716c in Okular 1.11.2 That commit looks like it'll solve keypresses being dropped, but doesn't look like it'll affect the obnoxious smooth scrolling effect. If you want to disable smooth scrolling entirely even though the slowness problem is now fixed, that's *yet another* separate issue. Currently (as of the git master version) you can do this by turning off animations globally in System Settings if you're using Plasma and just don't like any animations at all. If you're a person who does generally like some animations but just does not want animated scrolling only in Okular, then we plan to add an Okular-specific setting for that please see Bug 420755. *** Bug 421822 has been marked as a duplicate of this bug. *** (In reply to Nate Graham from comment #15) > Fixed with > https://invent.kde.org/graphics/okular/-/commit/ > e637e2ed519cb2dfe523981b2d6dc34a9855716c in Okular 1.11.2 I confirm that the problem goes away if the above patch is applied on okular-20.04.3. Excellent. (In reply to Nate Graham from comment #18) > If you want to disable smooth scrolling entirely even though the slowness > problem is now fixed, that's *yet another* separate issue. Currently (as of > the git master version) you can do this by turning off animations globally > in System Settings if you're using Plasma and just don't like any animations > at all. If you're a person who does generally like some animations but just > does not want animated scrolling only in Okular, then we plan to add an > Okular-specific setting for that please see Bug 420755. I would like to add my support for the Okular-specific setting to disable animated scrolling in Okular. I don't use Plasma so I still have the animations. Instant PgUp/PgDn navigation is ideal -- that is how I navigate in many different applications and having that impeded in Okular interrupts my workflow and is distracting. Another person to add support to be able to disable this horrible animation. Simple and quick jump scrolling is much better for my usage and helps save the planet saving all those unnecessary CPU and graphics chip processing cycles. You can disable it in Okular 20.12. Many thanks. I will build an RPM for our Fedora33 systems. |