<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/107132/">http://git.reviewboard.kde.org/r/107132/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On October 31st, 2012, 3:12 p.m. CET, <b>Dan Vrátil</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'd prefer to have a separate component right away, rather then hacking it now and waiting for someone else to clean it up.
One issue I've run to: I set Renkoo as group chat theme, reopened the Settings dialog, went to the Group chat tab again and the preview was not populated (this does not happen with SimKete though). I don't know whether this is caused by the Renkoo theme, but it might indicate problem in our code (I don't have any other themes to test right now). This does not happen in the "Single chat" tab.</pre>
</blockquote>
<p>On October 31st, 2012, 4:26 p.m. CET, <b>Daniele E. Domenichelli</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 didn't have any issue like that, does it happen every time and/or with any other theme?
I *think* that the Renkoo theme is broken for group chats though...</pre>
</blockquote>
<p>On November 1st, 2012, 12:23 a.m. CET, <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;">It's not "broken". It's just useless as you can't see who' sending you messages (especially if no-one has an avatar)</pre>
</blockquote>
<p>On November 1st, 2012, 12:09 p.m. CET, <b>Dan Vrátil</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 installed some additional themes and I was unable to reproduce it with any of them. And after changing "Single chat" theme, I was unable to reproduce it even with Renkoo, so I guess it's just some weird coincidental thingie.</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;">Should this review be discarded in favour of https://git.reviewboard.kde.org/r/108571/ ?</pre>
<br />
<p>- Dan</p>
<br />
<p>On November 10th, 2012, 12:22 a.m. CET, Daniele E. Domenichelli 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 Daniele E. Domenichelli.</div>
<p style="color: grey;"><i>Updated Nov. 10, 2012, 12:22 a.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;">This is a quick and dirty way to fix it, basically I duplicated the code for the single user chat and added a tabbed view.
Perhaps it could be a widget in order to avoid code duplication, but since we have just 2, I'd leave it like this for now, so that we can configure this in 0.6, and perhaps open a junior-job bug for this</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;">Config works, group chat works</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=282518">282518</a>,
<a href="http://bugs.kde.org/show_bug.cgi?id=295843">295843</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>config/appearance-config.ui <span style="color: grey">(7fe6ce43a952652edcca229b918283b05268b240)</span></li>
<li>lib/adium-theme-view.cpp <span style="color: grey">(24b9c3570699f64e1b751183430e4975dc6a449d)</span></li>
<li>config/appearance-config.h <span style="color: grey">(5b6c83d6ac4001f1d28b3ce1cb8e18a38e5269c5)</span></li>
<li>config/appearance-config.cpp <span style="color: grey">(7f42857998b509a03dc8168181850a7a9a2a477d)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107132/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>