<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D8056" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I gave this a shot and it's an excellent change--a huge improvement over the status quo. I have two suggestions for further ergonomic refinement:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">When the text field has focus, make the down arrow key move to the list of found result so that you can select one with the keyboard and not have to use the mouse at all</li>
<li class="remarkup-list-item">Pre-select the top result  in the list so that you can immediately hit the return key to select that application</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8056" rel="noreferrer">https://phabricator.kde.org/D8056</a></div></div><br /><div><strong>To: </strong>simgunz, dfaure, Frameworks, VDG, ngraham<br /><strong>Cc: </strong>ngraham, alexeymin, Frameworks<br /></div>