<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 />



 <p>Ship it!</p>



 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Looks so much better! You rock!</pre>
 <br />







<p>- Vishesh</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>