Summary: | Mouse cursor appearence does not change when hovering | ||
---|---|---|---|
Product: | [Applications] okular | Reporter: | Christian González <christian.gonzalez> |
Component: | general | Assignee: | Okular developers <okular-devel> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | aacid, corsaroangelo, elvis.angelaccio, oliver.sander |
Priority: | NOR | Keywords: | junior-jobs |
Version: | 0.19.0 | ||
Target Milestone: | --- | ||
Platform: | Ubuntu | ||
OS: | Linux | ||
Latest Commit: | https://commits.kde.org/okular/574fad92c8bed04e8b0c0fcbb84a9ecbd8cf685b | Version Fixed In: |
Description
Christian González
2014-05-15 08:52:33 UTC
*** Bug 311825 has been marked as a duplicate of this bug. *** Confirmed with current git master on Debian testing. Git commit 574fad92c8bed04e8b0c0fcbb84a9ecbd8cf685b by Albert Astals Cid, on behalf of Tobias Deiminger. Committed on 21/05/2017 at 21:37. Pushed by aacid into branch 'Applications/17.04'. Initialize cursor in PageView scrollbars with ArrowCursor REVIEW: 130131 M +2 -0 ui/pageview.cpp https://commits.kde.org/okular/574fad92c8bed04e8b0c0fcbb84a9ecbd8cf685b Missed the review. Would it be more correct to set the cursor on the viewport() instead of the whole scrollview? Then we do not need to fixup the scrollbars afterwards. It's not set on the whole scrollview, is it? PageView inherits from QAbstractScrollArea, so I am afraid the setCursor() calls there affect the complete scrollarea, not just the viewport. (In reply to Christoph Feck from comment #6) > PageView inherits from QAbstractScrollArea, so I am afraid the setCursor() > calls there affect the complete scrollarea, not just the viewport. Ah you were commenting on the original code and not on the bugfix. No idea, you can propose a fix, get someone to review and test it and then commit it if you think it's better :) |