<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/107048/">http://git.reviewboard.kde.org/r/107048/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KDE Base Apps and David Faure.</div>
<div>By Dawit Alemayehu.</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The attached patch is an attempt to address a rather difficult problem where attempting to open certain types of document in a background tab (MMB or CTRL+LMB) results in the activation of the part in that background tab.
The bug can be reproduced by simply attempting to open a PDF document using either the MMB or the CTRL+LMB button in Konqueror. If you have a local PDF file simply navigate to the location of the file and click on the file using the MMB. Otherwise, see the aforementioned bug report for a website link with PDF links in it. If the "Open new tabs in background" open is checked when you clicked on the PDF document, then the PDF document is opened embedded into a Konqueror tab. This background tab is then activated which causes it to obtain the focus when it should not. The consequence of the unintended activation of the background tab is the location bar changes its URL to the URL of the part in the background tab and the GUI is modified to reflect the contents of the part in the background tab.
All of that happens because the Okular part propagates the FocusIn event which is caught by KParts::Manager::eventFilter. It in turn invokes the virtual KParts::PartManager::setActivePart function which causes the unintended activation. This patch addressed the problem by ignoring any requests to active the tab so long as the reason for the activation is Qt::OtherFocusReason.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">- Tested opening a PDF file when the "Open new tabs in background" option is checked.
- Tested opening a PDF file when the "Open new tabs in background" option is UNchecked.
- Tested opening a PDF link from a webpage using both the MMB and CTRL+LMB.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=306417">306417</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>konqueror/src/konqviewmanager.cpp <span style="color: grey">(c8e3cb0)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107048/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>