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





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On March 30th, 2014, 1 p.m. CEST, <b>David Edmundson</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Where does this add it in the UI? Is it an optional button in the toolbar?
Maybe it should be in the menu too.

Thomas, any opinions? </pre>
 </blockquote>




 <p>On March 30th, 2014, 2:35 p.m. CEST, <b>Thomas Pfeiffer</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">First of all: What user story does this feature have? In which situation would users want to open the contact list from the chat window? Once we have that defined, we can think about how to implement it in the UI</pre>
 </blockquote>





 <p>On March 30th, 2014, 4:37 p.m. CEST, <b>Siddhartha Sahu</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">@David: Yes it is an optional button which the user will have to manually add it to the toolbar. Adding it to the menu sounds good too.

@Thomas: This patch is for https://bugs.kde.org/show_bug.cgi?id=307659.
Cases in which I think it will be helpful are when user is not using the desktop applet for some reason, or to quickly open the contact list from the chat window (eg using a shortcut).

Personally now that I'm using it, I find it very convenient being able to open the contact list (eg. after I receive a new message) without having to move the mouse all over to the system tray</pre>
 </blockquote>





 <p>On March 30th, 2014, 4:51 p.m. CEST, <b>Martin Klapetek</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I'm just curious - why do you open contact list after you receive new message?</pre>
 </blockquote>





 <p>On March 30th, 2014, 4:56 p.m. CEST, <b>Siddhartha Sahu</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Receiving a message from a friend asking about something, which I then ask to another friend ^_^</pre>
 </blockquote>





 <p>On March 30th, 2014, 5:35 p.m. CEST, <b>David Edmundson</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">For that would the following then be better:
 - A button "start new chat" in the menu
 - This opens a dialog with a list of all contacts
 - Selecting here starts a chat

?</pre>
 </blockquote>








</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">In other words, use KRunner? :)</pre>
<br />










<p>- Martin</p>


<br />
<p>On March 30th, 2014, 7:01 a.m. CEST, Siddhartha Sahu 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 Siddhartha Sahu.</div>


<p style="color: grey;"><i>Updated March 30, 2014, 7:01 a.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=313404">313404</a>


</div>



<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
ktp-text-ui
</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;">Will add documentation on userbase once committed :)</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;">Contact list opens</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>app/chat-window.cpp <span style="color: grey">(2c2e12b96620ad180f0269aff66fb860c0bb6fb5)</span></li>

 <li>app/chat-window.h <span style="color: grey">(793ac559cf3e640ca653dd60ba04c0fdbb264da5)</span></li>

</ul>

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







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








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