<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/100214/">http://git.reviewboard.kde.org/r/100214/</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;">Ok, better than previous, but with a lot of things to change, IMHO.
This thing has to work in the same way people changes desktop image background, grabbing it from the Internet.
So, just the "theme manager" has to be used, showing the "default" theme and eventually a couple of other ones. People has to see (and choice) theme screenshots, NOT compressed files where themes effectively are. And just ONE button to choose your theme, NOT the "Change theme" and the "Theme Manager" actions.
>From an user POV, I don't see the difference.</pre>
<br />
<p>- Andrea</p>
<br />
<p>On December 21st, 2010, 5:13 p.m., Furkan Üzümcü 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 rekonq.</div>
<div>By Furkan Üzümcü.</div>
<p style="color: grey;"><i>Updated 2010-12-21 17:13:02</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;">* Added support for changing Rekonq's new tab page theme.
* Added a Theme Manager</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;">* Tested by me and Panagiotis Papadopoulos and it works!</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>src/CMakeLists.txt <span style="color: grey">(f0310b4)</span></li>
<li>src/newtabpage.h <span style="color: grey">(be02d9d)</span></li>
<li>src/newtabpage.cpp <span style="color: grey">(7dcc958)</span></li>
<li>src/newtabpagetheme.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/newtabpagetheme.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/newtabpagetheme.ui <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/thememanager.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/thememanager.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/thememanager.ui <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100214/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>