<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/101924/">http://git.reviewboard.kde.org/r/101924/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Thanks for the update!
>> would only make sense to push it to the 4.7 branch.
> What exactly do you mean by 'only'? Isn't 4.7 the branch just
> about to be released and which will be in all distros until sometime next year?
I meant that this change should only be pushed to the KDE/4.7 and not to master as this class will be removed from master around the beginning of August (Dolphin 2.0...)</pre>
<br />
<p>- Peter</p>
<br />
<p>On July 20th, 2011, 2:23 p.m., Marcel Partap 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, David Faure and Peter Penz.</div>
<div>By Marcel Partap.</div>
<p style="color: grey;"><i>Updated July 20, 2011, 2:23 p.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;">What was strange that background highlighting and actual item selection were drawn independently from each other and the bogus highlighting to the left of the item was not cleared... Now this one was a __REAL__ bitch to get dealt with, took me hours and hours bashing my head against the shell ^^
ok now again the viewOptions is not only not the place to turn off background highlighting, but there it was even tried to ENABLE it :O
turned out this so called QStyle::SH_ItemView_ShowDecorationSelected documented as "When an item in an item view is selected, also highlight the branch or other decoration." is hard-coded on by DEFAULT in QCommonStyle and all inheriting from there so it requires a QProxyStyle to override the setting. While we have the opportunity, also set SH_ItemView_ArrowKeysNavigateIntoChildren for added joice of keyboard navigation (although strange effect comes up when being on a leaf and pressing Cursor::Right again - but with or without this setting, something with the selection handler...)
...now someone owes me CAKE for this one :D</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;">head-bashing</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>dolphin/src/views/dolphindetailsview.cpp <span style="color: grey">(0ce26df)</span></li>
<li>dolphin/src/views/dolphintreeview.h <span style="color: grey">(c037d41)</span></li>
<li>dolphin/src/views/dolphintreeview.cpp <span style="color: grey">(64b66aa)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101924/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/101924/s/195/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/07/11/dolphin-treeview-selection-paint-fail_400x100.png" style="border: 1px black solid;" alt="dolphin-treeview-selection-paint-fail" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>