<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/107796/">http://git.reviewboard.kde.org/r/107796/</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;">Sorry to be so nitpicking, but could we have at least the same color for the new icon? The color of the old icon fits in the current icon scheme, so this should be respected.</pre>
 <br />







<p>- Myriam</p>


<br />
<p>On January 7th, 2013, 10:37 a.m., Riccardo Ferrazzo 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 Amarok.</div>
<div>By Riccardo Ferrazzo.</div>


<p style="color: grey;"><i>Updated Jan. 7, 2013, 10:37 a.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;">I've replaced the svg queue icon with a painted green rounded rectangle that scale based on the digits to display</pre>
  </td>
 </tr>
</table>




<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>


 <a href="https://bugs.kde.org/show_bug.cgi?id=253802">253802</a>


</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>src/images/default-theme-clean.svg <span style="color: grey">(6bad334)</span></li>

 <li>src/images/default-theme.svg <span style="color: grey">(dbd3c90)</span></li>

 <li>src/playlist/view/listview/PrettyItemDelegate.cpp <span style="color: grey">(c13542b)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/107796/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/107796/s/918/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/18/snapshot1_400x100.png" style="border: 1px black solid;" alt="before" /></a>

 <a href="http://git.reviewboard.kde.org/r/107796/s/919/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/12/18/screenshot7_400x100.png" style="border: 1px black solid;" alt="after" /></a>

</div>


  </td>
 </tr>
</table>








  </div>
 </body>
</html>