<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/103401/">http://git.reviewboard.kde.org/r/103401/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit 58b38cdc042a7767d09ee952abeff7016dedd5dc by Dawit Alemayehu to branch master.</pre>
<br />
<p>- Commit</p>
<br />
<p>On December 13th, 2011, 7:37 a.m., Dawit Alemayehu wrote:</p>
<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>
<p style="color: grey;"><i>Updated Dec. 13, 2011, 7:37 a.m.</i></p>
<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 attach patch fixes the issue stated in the aforementioned bug report. More specifically if you split a directory view in Konqueror and attempt to preview a embeded inside of the view that does not have the focus, you get a very weird behavior that randomly affects both sides of the split view. For the description of the problem and how to reproduce it, see the bug report.
The fix was to simply comment out the code that attempt to hide the widget of the current view (inside KonqView::switchView) before switching out the dolphin part (filemanagement) for the part that is going to show the content to be previewed. When this piece of code is removed KonqViewManager::doSetActivePart is not somehow invoked with the current DolphinPart as the part to be set as the active one. Why that happens I do not know. However, after watching what happens when attempting to reproduce BR# 258278, I am tempted to say that this fix might actually resolve several other bug reports where clicking on a file to preview it in Konqueror did not result in the correct outcome. For example, see bug 279937, bug 280364 and bug 282283.</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=258278">258278</a>,
<a href="http://bugs.kde.org/show_bug.cgi?id=267215">267215</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/konqview.cpp <span style="color: grey">(f1e12b9)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103401/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>