<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/100835/">http://git.reviewboard.kde.org/r/100835/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/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 Network Management.</div>
<div>By Jirka Klimes.</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 patch fixes "Enable ..." checkbox handling and simplify the code.
There was a bug that when Wifi was rfkilled by a hardware switch,
knm unchecked the "Enable wireless" checkbox, but it also (erroneously)
issued 'disable wireless' command, which caused NM to store 'wireless
disabled' state as user preference. Then after enabling hardware switch
WiFi stayed disabled.
This is now fixed and the code is also simplified a bit.
Would you review please?
For Lamarque:
I think that b1c517b2b42a62990b6f9dd5b96eb79659e9953b (saving config) is not necessary and may be contra-productive.
NM itself stores user preference and there's no need applets do that. If there are more applets they will step on their toes.
And in NM 0.9 will be perfectly possible to have more applets running.
Nonetheless, thanks for your work on knm!</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>applet/nmpopup.cpp <span style="color: grey">(282299b)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100835/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>