<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="https://git.reviewboard.kde.org/r/118644/">https://git.reviewboard.kde.org/r/118644/</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;">Thank you very much!</pre>
 <br />









<p>- Kai Uwe Broulik</p>


<br />
<p>On June 10th, 2014, 5:09 p.m. UTC, Andrew Lake wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for Plasma and Kai Uwe Broulik.</div>
<div>By Andrew Lake.</div>


<p style="color: grey;"><i>Updated June 10, 2014, 5:09 p.m.</i></p>









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


<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;">Update battery monitor to use new icons from new battery.svgz theme file in popup.

Also includes minor updates to vertical alignments of icon with text and slight increase in battery status text opacity.

(Requires replacing the battery.svgz theme file in the theme icons/ directory with the new one attached - I wasn't sure how to do a review request across two different repos)</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>applets/batterymonitor/contents/ui/BatteryIcon.qml <span style="color: grey">(308eaf85c8d41574f210086b6ce17a47e261a773)</span></li>

 <li>applets/batterymonitor/contents/ui/BatteryItem.qml <span style="color: grey">(655026b83333e5a429f70e55aa8b9d01218139ac)</span></li>

 <li>applets/batterymonitor/contents/ui/BrightnessItem.qml <span style="color: grey">(0d7e8e2b4a2acae4e5a2ab43a9edefc812216ee6)</span></li>

 <li>applets/batterymonitor/contents/ui/PopupDialog.qml <span style="color: grey">(44f6f9dd55d6356c446f4c7352bc5bc03338d870)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/118644/diff/" style="margin-left: 3em;">View Diff</a></p>



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>

<ul>

 <li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2014/06/10/1b839cb5-f507-42c0-9314-cd9e0a17ee1f__battery.svgz">New battery.svgz theme file</a></li>

 <li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2014/06/10/6bb20e04-22fd-4ad6-bff6-9afee5cfd4de__iconsreviewrequestss.png">SS from my laptop</a></li>

 <li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2014/06/10/102356b1-0d85-45fa-96de-93384e444433__mockup_battery_monitor_2.png">icon assets shown in mockup</a></li>

</ul>





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








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