[Okular-devel] Review Request 120660: Allow each PageView to use a different tool

Jonathan Doman jonathan.doman at gmail.com
Wed Dec 10 14:34:45 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120660/
-----------------------------------------------------------

(Updated Dec. 10, 2014, 2:34 p.m.)


Status
------

This change has been marked as submitted.


Review request for Okular.


Bugs: 334251
    http://bugs.kde.org/show_bug.cgi?id=334251


Repository: okular


Description
-------

Keep a local MouseMode setting, and don't rely on the value returned by Settings::mouseMode().

In the bug report, I stated that each tab should be allowed to use different tools. No one objected, but after testing it's not clear this is the right answer. The existing behavior was to force all tabs/windows to use the same tool, but then it's not clear what to do with review mode.


Diffs
-----

  ui/pageview.cpp 17e66f4a3b420bbcaf281532fa9d84379c74d48c 

Diff: https://git.reviewboard.kde.org/r/120660/diff/


Testing
-------

Followed steps in bug report for both tabs and windows to make sure different tools are allowed.


Thanks,

Jonathan Doman

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20141210/258c2543/attachment.html>


More information about the Okular-devel mailing list