<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/108874/">http://git.reviewboard.kde.org/r/108874/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://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 Solid and Dario Freddi.</div>
<div>By Àlex Fiestas.</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;">External monitors brightness can't be change, so the setting is useless in all workstations.
Keyboard brightness is useless for those laptop without keyboard backlight.
This patch hide (do not load) these settings for devices that do not support them.</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;">KCM does not show brightness/keyboard options in my workstation.</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>powerdevil/daemon/actions/bundled/brightnesscontrol.h <span style="color: grey">(a081340)</span></li>
<li>powerdevil/daemon/actions/bundled/brightnesscontrol.cpp <span style="color: grey">(323905d)</span></li>
<li>powerdevil/daemon/actions/bundled/keyboardbrightnesscontrol.h <span style="color: grey">(13fd01c)</span></li>
<li>powerdevil/daemon/actions/bundled/keyboardbrightnesscontrol.cpp <span style="color: grey">(8249b17)</span></li>
<li>powerdevil/daemon/actions/bundled/powerdevilbrightnesscontrolaction.desktop <span style="color: grey">(e9db51f)</span></li>
<li>powerdevil/daemon/actions/bundled/powerdevilkeyboardbrightnesscontrolaction.desktop <span style="color: grey">(3a4392a)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/108874/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>