<table><tr><td style="">sander created this revision.<br />sander added a reviewer: Okular.<br />Herald added a project: Okular.<br />Herald added a subscriber: okular-devel.<br />sander requested review of this revision.
</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/D15123">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Currently these tooltips only appear in 'Browse' mode.  KDE bug <a href="https://bugs.kde.org/show_bug.cgi?id=397783" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=397783</a> asks for them to be shown in 'text selection' mode, too.  I don't really see a reason to disable them in any mode, and therefore this patch simply removes the check for the mouse mode completely.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>1 Open a pdf file with an internal or hyperlink<br />
2 Switch to Browse mode<br />
3 Hover over the link: a tooltip with the link target appears<br />
4 Switch to text selection, zoom, magnifier, or any other mouse mode<br />
5 Repeat Step 3: it still works</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15123">https://phabricator.kde.org/D15123</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>ui/pageview.cpp</div></div></div><br /><div><strong>To: </strong>sander, Okular<br /><strong>Cc: </strong>okular-devel, ngraham, aacid<br /></div>