<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/117542/">https://git.reviewboard.kde.org/r/117542/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#e0e0e0" width="100%" cellpadding="8" style="border: 1px gray solid;">
<tr>
<td>
<h1 style="margin-right: 0.2em; padding: 0; font-size: 10pt;">This change has been discarded.</h1>
</td>
</tr>
</table>
<br />
<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.</div>
<div>By Kai Uwe Broulik.</div>
<p style="color: grey;"><i>Updated June 19, 2014, 7:56 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;">This adds the following improvements provided by Review 117339 and Review 117540:
- Show temperature, voltage, energy rate and serial number. Since those are only shown when expanding the battery it's imho not that bad to show some geeky stuff which can give you some more indication of battery health and overall power consumption. It adds new strings, however. It also changes the substr hackery to "enabled: false" for greying out the labels.
What's the preferred way of formatting numbers (decimal points, units)? The formats thing only seems to be for file sizes and durations.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Compiles. Information is properly displayed when expanding a battery.</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/code/logic.js <span style="color: grey">(5b4cb52)</span></li>
<li>applets/batterymonitor/contents/ui/BatteryItem.qml <span style="color: grey">(431aa9a)</span></li>
<li>applets/batterymonitor/contents/ui/CompactRepresentation.qml <span style="color: grey">(39cd331)</span></li>
<li>applets/batterymonitor/contents/ui/DetailsLabel.qml <span style="color: grey">(d973d22)</span></li>
<li>applets/batterymonitor/contents/ui/PopupDialog.qml <span style="color: grey">(0444199)</span></li>
<li>applets/batterymonitor/contents/ui/batterymonitor.qml <span style="color: grey">(4711c5e)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/117542/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>