[okular] [Bug 421437] cursor is still a pointing hand when scrolling stops after clicking an internal link

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=421437

Nate Graham <nate at kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/grap |https://invent.kde.org/grap
                   |hics/okular/commit/8b008b02 |hics/okular/commit/843c43cc
                   |fc31e20f7c74ab2c25d34160756 |6ba82a80db86e0a5ed11e27e255
                   |c7a92                       |cd851

--- Comment #6 from Nate Graham <nate at kde.org> ---
Git commit 843c43cc6ba82a80db86e0a5ed11e27e255cd851 by Nate Graham, on behalf
of David Hurka.
Committed on 21/10/2020 at 00:52.
Pushed by ngraham into branch 'master'.

Fix cursor not updated after clicking internal link

updateCursor() was called by wheelEvent(), which made sense,
because after the wheel event the page will have moved under the cursor.

With smooth scrolling, it makes less sense in wheelEvent(),
because at that point scrolling is still in the future.

scrollContentsBy() appears to be called on every scroll step.
(It is documented to be called at scrollbar value changes, so makes sense.)

This patch removes updateCursor() from wheelEvent(), but adds it to
scrollContentsBy().

I did not check anything out with d->visibleItems, as was indicated it
graphics/okular!176.

M  +0    -4    autotests/parttest.cpp
M  +5    -2    ui/pageview.cpp

https://invent.kde.org/graphics/okular/commit/843c43cc6ba82a80db86e0a5ed11e27e255cd851

-- 
You are receiving this mail because:
You are the assignee for the bug.