<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 />
<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 2d97851b8ad9991bffb05419fbde3d9888484ce1 by Andrew Lake to branch master.</pre>
<br />
<p>- Commit Hook</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>