<table><tr><td style="">simgunz added a comment.
</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><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D8056#158939" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D8056#158939</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p><a href="https://phabricator.kde.org/p/simgunz/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@simgunz</a>, have you had a chance to try implementing David's suggestion?</p></div>
</blockquote>

<p>Yes. Everything seems to work but I haven't pushed yesterday because I wanted to do an extra test.</p>

<p>The current behavior is the following:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Down arrow pass the focus to the QTreeView and moves the selection down by one. (alternatively we can just pass the focus without changing the selection).</li>
<li class="remarkup-list-item">Up arrow is used to go back in the history of the combobox (as the current behavior)</li>
<li class="remarkup-list-item">Making down arrow change the focus, partially breaks it functionality of navigating the history of the combo box.</li>
<li class="remarkup-list-item">If the filter doesn't match anything, down arrow doesn't change the focus and works for navigating the history</li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">The feature is not enabled if PopupAutocomplete is enabled. Theoretically it should only be disabled while the popup is shown, but I couldn't find a way to detect this state.</li>
</ul>

<p>Moreover the first match is selected by default and hitting enter calls it. To enable this feature I had to change the behaviour so that the text box is not filled anymore with the command corresponding to the selected desktop entry. For me this is even cleaner from the UI point of view.</p>

<p>I'll push tomorrow evening.</p></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>