<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/104226/">http://git.reviewboard.kde.org/r/104226/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On April 17th, 2012, 11:18 a.m., <b>Kai Uwe Broulik</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I just applied it to latest master and found a few usability issues:
* The settings dialog that contains all the configs is way too small
* The slider "grab" is 10 pixels to the left, some QML component issue
And the showstopper issue is:
There is no easy way to see the battery percentage. The overlay number is no longer available. And also there is no tooltip, i.e. hovering the plasmoid also does nothing. You _always_ have to click the battery plasmoid to see the exact percentage.
The tooltip thing seems to be a general QML plasmoid issue, the new devicec notifier also lacks one.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The patch is not up-to-date. Please checkout the branch plasma/viranch/powermanagementservices and compile plasma/generic/{applets/batterymonitor,dataengines/powermanagement}. That's where all my latest commits have gone.</pre>
<br />
<p>- Viranch</p>
<br />
<p>On April 3rd, 2012, 5:12 a.m., Viranch Mehta 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 Plasma.</div>
<div>By Viranch Mehta.</div>
<p style="color: grey;"><i>Updated April 3, 2012, 5:12 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 fixed the QML battery monitor to be fairly usable and this diff merges it to master.</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;">Applet and dataengine both tested and work fine.</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>plasma/generic/applets/CMakeLists.txt <span style="color: grey">(2dedcb2)</span></li>
<li>plasma/generic/applets/batterymonitor/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasma/generic/applets/batterymonitor/Messages.sh <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasma/generic/applets/batterymonitor/README.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasma/generic/applets/batterymonitor/battery-oxygen-inkscape.svgz <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasma/generic/applets/batterymonitor/battery-oxygen.svgz <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasma/generic/applets/batterymonitor/contents/config/main.xml <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasma/generic/applets/batterymonitor/contents/ui/IconButton.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasma/generic/applets/batterymonitor/contents/ui/PopupDialog.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasma/generic/applets/batterymonitor/contents/ui/config.ui <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasma/generic/applets/batterymonitor/metadata.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasma/generic/dataengines/powermanagement/powermanagementengine.h <span style="color: grey">(20642c2)</span></li>
<li>plasma/generic/dataengines/powermanagement/powermanagementengine.cpp <span style="color: grey">(5572fcb)</span></li>
<li>plasma/generic/dataengines/powermanagement/powermanagementjob.h <span style="color: grey">(2c32015)</span></li>
<li>plasma/generic/dataengines/powermanagement/powermanagementjob.cpp <span style="color: grey">(e205bb0)</span></li>
<li>plasma/generic/dataengines/powermanagement/powermanagementservice.operations <span style="color: grey">(ad1301f)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104226/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/104226/s/487/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/03/23/battery1_400x100.png" style="border: 1px black solid;" alt="with top right graphic in the popup" /></a>
<a href="http://git.reviewboard.kde.org/r/104226/s/503/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/04/03/battery2_400x100.png" style="border: 1px black solid;" alt="applet on the workspace" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>