<table><tr><td style="">sander updated this revision to Diff 49288.<br />sander edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-iewap2yiis55taf/">(Show Details)</a><br />sander edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-i2zvruge3c7qdmh/">(Show Details)</a><br />sander added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D18118">View Revision</a></tr></table><br /><div><div><p>New patch version:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">You can go back by touching on the left half of a touchscreen, touching the right half will go forward</li>
<li class="remarkup-list-item">Mouse behavior is unchanged---left-clicking anywhere on the screen will go forward.</li>
</ul></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">Previously, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">left-click</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">touch</span>ing <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">with the mouse</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">a touch screen</span> would advance the presentation to the next page no matter the cursor position.  With this patch, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">clicks on</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">touching</span> the right half of the screen still lead<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">s</span> to the next page being shown, but <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">clicks on</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">touching</span> the left half lead<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">s</span> to the previous page now. <br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">As touch events are set to create mouse events, touching on the left half of a touch screen will lead to the previous page, too. This should make people happy that use Okular for sheet music.<br />
<br />
</span>The actual action happens on mouse-release.  For touch screens this means less interference with swipe gestures (which also allow to go to the previous/next page).<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">* Open a pdf in presentation mode<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Click on</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">* Touch</span> the right half of the <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">screen (with the left mouse button)</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">touchscreen</span><br />
* Result: the next page is shown<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Click on</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">* Touch</span> the left half of the screen<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D18118?vs=49035&id=49288">https://phabricator.kde.org/D18118?vs=49035&id=49288</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18118">https://phabricator.kde.org/D18118</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>ui/presentationwidget.cpp<br />
ui/presentationwidget.h</div></div></div><br /><div><strong>To: </strong>sander, Okular<br /><strong>Cc: </strong>gbodley, michaelweghorn, aacid, ngraham, okular-devel, darcyshen<br /></div>