<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="http://git.reviewboard.kde.org/r/110545/">http://git.reviewboard.kde.org/r/110545/</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;">This review has been submitted with commit 62bb42b610fc4d9228ed77a61dbef143658d8340 by Dan Vrátil to branch master.</pre>
 <br />









<p>- Commit</p>


<br />
<p>On May 20th, 2013, 1:30 p.m. UTC, Dan Vrátil wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://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 Dan Vrátil.</div>


<p style="color: grey;"><i>Updated May 20, 2013, 1:30 p.m.</i></p>






<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;">The chat UI is reinitialized when theme is changed, the current conversations are reloaded from TpLogger.

I've ported LogManager to LogWalker, which simplifies and speeds up the code considerably. That means that text-ui now depends on TelepathyLoggerQt 0.8.0 (0.7.95) and TelepathyLogger 0.8.0 - make sure you have latest git build of TpLoggerQt otherwise you get crash on text-ui start. If everything works, I'll make TpLoggerQt 0.8.0 release soon (screw waiting for FD.O account).

Status notifications are not stored in logger, so after re-loading the chat they disappear, but I don't think that is a problem.</pre>
  </td>
 </tr>
</table>




<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=270350">270350</a>


</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>CMakeLists.txt <span style="color: grey">(bb16e0b)</span></li>

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

 <li>app/chat-window.cpp <span style="color: grey">(3a9b6e9)</span></li>

 <li>config/appearance-config.h <span style="color: grey">(c0a1e69)</span></li>

 <li>config/appearance-config.cpp <span style="color: grey">(f7ebd40)</span></li>

 <li>lib/chat-widget.h <span style="color: grey">(366d88b)</span></li>

 <li>lib/chat-widget.cpp <span style="color: grey">(39da92c)</span></li>

 <li>lib/logmanager.h <span style="color: grey">(e5e97bc)</span></li>

 <li>lib/logmanager.cpp <span style="color: grey">(213b5aa)</span></li>

</ul>

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







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








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