<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/108802/">http://git.reviewboard.kde.org/r/108802/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.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 Frank Reininghaus.</div>
<div>By Dawit Alemayehu.</div>
<p style="color: grey;"><i>Updated May 13, 2013, 1:02 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 is the updated diff based on your feedback.
There is only one slight behavioral change because of the removal of the parameter from update that allowed us to simulate the shiftKeyPressed state. I do not think this behavior change is worth moving the code that checked for Shift key press into the calling code is worth it. Anyhow, the behavior change is if a user presses both Shift keys while the context menu is visible, lets one of them go and presses it again. The action no longer changes where as before it did. To make it clear here is a description of what the behavior was previously:
- Press one Shift key, action changes to "Delete"
- Press the second Shift key while still holding down the first Shift key and the action goes back "Move to Trash"
- Release the second Shift key, nothing changes.
- Press the second shift key again and the action changes to "Delete".
The difference is that last behavior. With this code change, pressing the second Shift key does nothing until you release the first Shift key.
</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;">This patch fixes DolphinPart such that the "Delete/Move To Trash" actions are automatically toggled if the user presses the Shift key and allows https://git.reviewboard.kde.org/r/107509/ to be applied.
The code is completely based on what Dolphin's context menu does. Even though this works as planned, I still have reservations about the use of KModifierKeyInfo since every key press event from any application is sent to the application that connects to its signals. In my code and unlike what is done in Dolphin's context menu, I try to mitigate the impact of that by ignoring the signal when the part does not have the focus. Still if there is a better way to capture key press events at the part level I would like to use that instead. Any ideas ?</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">(ffb232c)</span></li>
<li>dolphin/src/dolphincontextmenu.h <span style="color: grey">(1c65fab)</span></li>
<li>dolphin/src/dolphincontextmenu.cpp <span style="color: grey">(89a169f)</span></li>
<li>dolphin/src/dolphinpart.h <span style="color: grey">(7881ded)</span></li>
<li>dolphin/src/dolphinpart.cpp <span style="color: grey">(627ba79)</span></li>
<li>dolphin/src/dolphinremoveaction.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>dolphin/src/dolphinremoveaction.cpp <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/108802/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>