<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/106123/">https://git.reviewboard.kde.org/r/106123/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#e0e0e0" width="100%" cellpadding="12" style="border: 1px gray solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<h1 style="margin: 0; padding: 0; font-size: 10pt;">This change has been discarded.</h1>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Plasma, Martin Gräßlin and Lamarque Souza.</div>
<div>By Richard Stockton.</div>
<p style="color: grey;"><i>Updated Aug. 31, 2015, 9:31 p.m.</i></p>
<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=297842">297842</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kde-workspace
</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;">Fix for Kickoff-Widget Keyboard Navigation enhancement (review request 104422): Following enhancement, when a 'search' is performed, the UI became unable to move among the views. This patch enables Qt::Key_Tab to (a) enter into the search Results View, where a program may be selected; (b) move back into the search bar TextEdit widget, by pressing Key_Tab twice; and (c) re-expose the other views, by pressing Key_Tab once (highlighting all text within the seearch bar TextEdit widget), and then pressing Key_Delete to empty the TextEdit field. When the textEdit is empty, the flipscrollview (e.g., "Applications") replaces the Search Results View, and the User is back to the Initial State of the Kickoff-Widget GUI.</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;">1: search with multiple search results: Tab enters the results view, and arrow keys can choose from multiple results. Key_Enter does run the selected Application. Per above, Key_Tab returns to the search TextEdit, with existing contents pre-selected/Highlighted.
2. search with empty results: arrow keys do nothing. Tab key returns to the search TextEdit, as in Test #1.
3. When search TextEdit contains no text (via Key_Delete when highlighted, or pressing Key_Delete multiple times) the Initial State of the views is restored: (Tab Bar, Serch Bar, and the selected Tab content present and executable.
4. Key_Escape closes the Kickoff UI, without executing a program.
5. Inability to use Up/Down arrows to go between Views is unfortunate, but remains unchanged from before.</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>plasma/desktop/applets/kickoff/ui/launcher.cpp <span style="color: grey">(902e121)</span></li>
<li>plasma/desktop/applets/kickoff/ui/searchbar.cpp <span style="color: grey">(6937f93)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/106123/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>