[Okular-devel] Review Request: Better next/previous page behaviour for facing pages

Thomas Etter etterth at eth.edu
Wed Apr 18 17:56:10 UTC 2012


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

(Updated April 18, 2012, 5:56 p.m.)


Review request for Okular and Albert Astals Cid.


Changes
-------

I now use the left/right arrow keys if there is no horizontal scrollbar to change the current page.

This should be accessibility compliant and is much nicer than my previous patch. Thanks for the feedback.


Description
-------

If two pages can be displayed at the same time while using a Facing mode, move two pages when pressing "next page" or "previous page".


Diffs (updated)
-----

  ui/pageview.cpp 4be5e6b 

Diff: http://git.reviewboard.kde.org/r/104596/diff/


Testing
-------

Works on my system


Thanks,

Thomas Etter

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


More information about the Okular-devel mailing list