<table><tr><td style="">sredman created this revision.<br />sredman added a reviewer: KDE Connect.<br />Herald added a project: KDE Connect.<br />Herald added a subscriber: kdeconnect.<br />sredman requested review of this revision.
</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/D17614">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Allow filter box to handle keystrokes passed to the ListView<ul class="remarkup-list">
<li class="remarkup-list-item">This is a little cheesy, but fixes <a href="https://phabricator.kde.org/T8341" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T8341</a></li>
</ul></li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">Pin filter box to always be visible</li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">Clear filter on Esc</li>
</ul></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Try typing a contact you would like to search for<ul class="remarkup-list">
<li class="remarkup-list-item">Before: Every time you type a new letter, the filter box would lose focus and you would have to click it to give it focus again</li>
<li class="remarkup-list-item">Now: You can type smoothly</li>
</ul></li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">Try scrolling the list<ul class="remarkup-list">
<li class="remarkup-list-item">Before: The filter box disappeared off the top of the screen</li>
<li class="remarkup-list-item">Now: The filter box is always visible</li>
</ul></li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">Try pressing escape<ul class="remarkup-list">
<li class="remarkup-list-item">Before: Nothing</li>
<li class="remarkup-list-item">Now: Filter clears</li>
</ul></li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">Bonus: When using the arrow keys to navigate, before the filter was manually handling those and the view would not loop. Now, the view loops if you try to select an item past the beginning or end (Easiest to see with highlighting from <a href="https://phabricator.kde.org/D17612" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D17612</a>)</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R224 KDE Connect</div></div></div><br /><div><strong>BRANCH</strong><div><div>smsapp-filter</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17614">https://phabricator.kde.org/D17614</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>smsapp/qml/ConversationList.qml</div></div></div><br /><div><strong>To: </strong>sredman, KDE Connect<br /><strong>Cc: </strong>kdeconnect, mschroeder, varunp, shivanshukantprasad, skymoore, brute4s99, wistak, anoopv, dvalencia, rmenezes, julioc, Leptopoda, timothyc, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, mikesomov, tctara, apol<br /></div>