<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/102987/">http://git.reviewboard.kde.org/r/102987/</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;">> As the title says. There will be more changes of the UI in later commits, so it wouldn't make much sense to discuss now about the look of the achievements tab in the player.
I would personally like to see the mocup of the imagined Ui before many patches coming. I would like to agree upon something before getting any new Ui in and code then to maintain (On the other hand, the whole Ui interface of this player should be redesigned by a person with relevant skills - for future reference).</pre>
<br />
<p>- Laszlo</p>
<br />
<p>On November 4th, 2011, 6:45 p.m., Felix Rohrbach wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/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 Gluon.</div>
<div>By Felix Rohrbach.</div>
<p style="color: grey;"><i>Updated Nov. 4, 2011, 6:45 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;">As the title says. There will be more changes of the UI in later commits, so it wouldn't make much sense to discuss now about the look of the achievements tab in the player.</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>engine/achievement.cpp <span style="color: grey">(021ee24)</span></li>
<li>engine/achievementsmanager.h <span style="color: grey">(1e9e24d)</span></li>
<li>engine/achievementsmanager.cpp <span style="color: grey">(e08a05d)</span></li>
<li>player/desktop/CMakeLists.txt <span style="color: grey">(01cd3ee)</span></li>
<li>player/desktop/delegates/achievementdelegate.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>player/desktop/delegates/achievementdelegate.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>player/desktop/gamedetailsoverlay.h <span style="color: grey">(dd95f2a)</span></li>
<li>player/desktop/gamedetailsoverlay.cpp <span style="color: grey">(ca0d6a8)</span></li>
<li>player/lib/models/achievementsmodel.h <span style="color: grey">(c25c1f9)</span></li>
<li>player/lib/models/achievementsmodel.cpp <span style="color: grey">(34000ff)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102987/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>