<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="https://git.reviewboard.kde.org/r/117991/">https://git.reviewboard.kde.org/r/117991/</a>
     </td>
    </tr>
   </table>
   <br />





 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Thanks for the patch; it's a good start but however not very good approach, we use a filter proxy model which can do the filtering for us, so the string should just be passed there.

In any case, see https://git.reviewboard.kde.org/r/117916/ where someone actually has done this already.

It also helps if you talk to us first about your plans before actual coding, we hate seeing code being thrown away because it was already done...sorry :)
</pre>
 <br />









<p>- Martin Klapetek</p>


<br />
<p>On May 4th, 2014, 9:56 p.m. CEST, Olaf Piesche wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://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 Telepathy.</div>
<div>By Olaf Piesche.</div>


<p style="color: grey;"><i>Updated May 4, 2014, 9:56 p.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
ktp-contact-list
</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;">When typing with the contact list widget in focus, finds the first matching contact to the string entered and selects it. If more than 1s passes between keyboard events, the search string is cleared. The only way I could figure out how to do this is to iterate over the contactsModel and match and select manually.</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;">Typing with the contact list focused to find contacts. Tried other keyboard shortcuts, which still work. </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>contact-list-widget.h <span style="color: grey">(8f39b81)</span></li>

 <li>contact-list-widget.cpp <span style="color: grey">(3ffa1cc)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/117991/diff/" style="margin-left: 3em;">View Diff</a></p>







  </td>
 </tr>
</table>








  </div>
 </body>
</html>