-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/130131/
-----------------------------------------------------------
(Updated May 21, 2017, 9:37 p.m.)
Status
------
This change has been marked as submitted.
Review request for Okular.
Changes
-------
Submitted with commit 574fad92c8bed04e8b0c0fcbb84a9ecbd8cf685b by Albert Astals
Cid on behalf of Tobias Deiminger to branch Applications/17.04.
Bugs: 334798
http://bugs.kde.org/show_bug.cgi?id=334798
Repository: okular
Description
-------
QT Docs [say](http://doc.qt.io/qt-5/qwidget.html#cursor-prop): "If no cursor
has been set, or after a call to unsetCursor(), the parent's cursor is used."
For us this means, as long as PageView child scrollbars are not initialized,
their cursor will be whatever the PageView cursor is. I think it's reasonable
to let the scrollbar cursors always be an arrow, unconditionally.
Furthermore, would you expect the cursor in selection mode to change from cross
back to arrow if it resides in the "empty grey space" in between the page and
the scrollbar? At the moment the patch doesn't change that behaviour, cursor
will stay a cross there.
Diffs
-----
ui/pageview.cpp 28cf77df9d92b92a33f427217a1438bc452c0eb0
Diff: https://git.reviewboard.kde.org/r/130131/diff/
Testing
-------
Thanks,
Tobias Deiminger
