<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/100849/">http://git.reviewboard.kde.org/r/100849/</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 13th, 2011, 9:15 p.m., <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;">You can't do it like this. Sorry.
We deliberately support pretty much every theme from the Adium website, therefore we can't modify any theme code to make it fit what we want - as we'd have to do it for all of them which doesn't make sense.
Besides: Fully customisable background support is both planned and do-able.
I think [citation needed] each themes .plist file also includes a flag that says whether it's a good idea to have configurable backgrounds or not.
Also if we just matched background colours if you have light coloured Chat Theme with dark KDE colours, us making a combined mix of them could look horrendous.</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;">Ok, I get your point. It was a kind of test. In fact I wasn't even sure I could mess with the "main.css" file. Now that I think about it, you're right about the themes, if someone has a dark theme he/she should download a matching theme from audium and viceversa.
Ok, delete this review.</pre>
<br />
<p>- Francesco</p>
<br />
<p>On March 13th, 2011, 2:31 p.m., Francesco Nwokeka wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Telepathy.</div>
<div>By Francesco Nwokeka.</div>
<p style="color: grey;"><i>Updated March 13, 2011, 2:31 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;">Fix for bug 268281 @ https://bugs.kde.org/show_bug.cgi?id=268281
the default color was set by the css that was "overwriting" any customization made by c++.</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;">Changed color of my kde theme and the chat window responded accordingly</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>data/styles/renkoo.AdiumMessageStyle/Contents/Resources/main.css <span style="color: grey">(c2ce2d5)</span></li>
<li>lib/adium-theme-view.cpp <span style="color: grey">(1eefafb)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100849/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>