<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/117916/">https://git.reviewboard.kde.org/r/117916/</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;">As you know from bugzilla, this was slihghtly controversial. There's some good args for and against. I'm going to try it for a bit and report back and some real world usage.</pre>
 <br />









<p>- David Edmundson</p>


<br />
<p>On April 30th, 2014, 9:09 p.m. UTC, Renato Atilio 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 Renato Atilio.</div>


<p style="color: grey;"><i>Updated April 30, 2014, 9:09 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=318591">318591</a>


</div>



<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;">Replace a void action (typing does absolutely nothing currently) by the contact filtering when typing.

What this diff does:
1. Focus the contacts list tree view instead of the status combo box on the contact list opening
2. When searching, focus the contacts list and select the index from keyboardSerch() if valid or first index otherwise - so the user can just type Enter after searching, if needed, to open the chat window
3. When typing on the contacts list, it captures the key events and sends them to the filter bar so it can determine if the event is a text insertion, a backspace or a close request
4. Set the focus on the contacts list after a close request</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">(9aa3a48)</span></li>

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

 <li>filter-bar.h <span style="color: grey">(76dc572)</span></li>

 <li>filter-bar.cpp <span style="color: grey">(dd57421)</span></li>

 <li>main-widget.h <span style="color: grey">(aa69daa)</span></li>

 <li>main-widget.cpp <span style="color: grey">(27a941d)</span></li>

</ul>

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







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








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