<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/107516/">http://git.reviewboard.kde.org/r/107516/</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 439fa9964245400d8c3ac888ddea18dcd9382754 by Aurélien Gâteau to branch master.</pre>
<br />
<p>- Commit</p>
<br />
<p>On November 29th, 2012, 4:10 p.m., Aurélien Gâteau 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 Nepomuk and Vishesh Handa.</div>
<div>By Aurélien Gâteau.</div>
<p style="color: grey;"><i>Updated Nov. 29, 2012, 4:10 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;">This patch prettifies Nepomuk StatusWidget. I made the following changes:
- Reorganize the widget content so that status text won't be cropped anymore. Set the label of the status text to wordwrap to ensure it is never truncated.
- Remove the bold on index count. Reason: it looks too big IMO.
- Get rid of KTitleWidget, replace it with a label and a bigger icon. Reason: it looks weird because the left of its text is not aligned with the other text below it.
Changes do not introduce any new strings, as can be seen in the screenshots (fully french translated)</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>nepomuk/kcm/statuswidget.cpp <span style="color: grey">(a5b7ae8)</span></li>
<li>nepomuk/kcm/statuswidget.ui <span style="color: grey">(c518300)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107516/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/107516/s/860/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/11/29/nepomukcontroller_400x100.png" style="border: 1px black solid;" alt="before-after" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>