<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/102450/">http://git.reviewboard.kde.org/r/102450/</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 Peter Penz.</div>
<div>By Tirtha Chatterjee.</div>
<p style="color: grey;"><i>Updated Aug. 28, 2011, 11:33 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">Here's a new diff, after the design changes we discussed over mail. It incorporates all of them.</pre>
</td>
</tr>
</table>
<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;">Allow opening files and directories by pressing 'Enter' key. In case multiple files are selected when enter is pressed, all of them are opened. In case of multiple directories, the first directory gets opened in the current tab, while the other directories open in new tabs.</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;">Yes, tested and working.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>dolphin/src/CMakeLists.txt <span style="color: grey">(b443aa7)</span></li>
<li>dolphin/src/kitemviews/kfileitemmodel.h <span style="color: grey">(0fecbcf)</span></li>
<li>dolphin/src/kitemviews/kfileitemmodel.cpp <span style="color: grey">(c2f49f7)</span></li>
<li>dolphin/src/kitemviews/kitemlistcontainer.cpp <span style="color: grey">(0d2637d)</span></li>
<li>dolphin/src/kitemviews/kitemlistcontroller.h <span style="color: grey">(04d4985)</span></li>
<li>dolphin/src/kitemviews/kitemlistcontroller.cpp <span style="color: grey">(207535c)</span></li>
<li>dolphin/src/kitemviews/kitemlistkeyboardsearchmanager.cpp <span style="color: grey">(34633d6)</span></li>
<li>dolphin/src/kitemviews/kitemlistkeyboardsearchmanager_p.h <span style="color: grey">(cf41f3b)</span></li>
<li>dolphin/src/kitemviews/kitemlistview.h <span style="color: grey">(9977e36)</span></li>
<li>dolphin/src/kitemviews/kitemlistview.cpp <span style="color: grey">(a35752a)</span></li>
<li>dolphin/src/kitemviews/kitemmodelbase.h <span style="color: grey">(c4e0464)</span></li>
<li>dolphin/src/kitemviews/kitemmodelbase.cpp <span style="color: grey">(69f62bc)</span></li>
<li>dolphin/src/views/dolphinview.h <span style="color: grey">(437f12f)</span></li>
<li>dolphin/src/views/dolphinview.cpp <span style="color: grey">(959e4da)</span></li>
<li>plasma/applets/folderview/plasma-applet-folderview.desktop <span style="color: grey">(03fbfff)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102450/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>