<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://svn.reviewboard.kde.org/r/6041/">http://svn.reviewboard.kde.org/r/6041/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On December 4th, 2010, 5:29 p.m., <b>Aaron Seigo</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 think it does need a string to explain, because unless one understands _why_ it is doing such a thing it could appear pretty &quot;magical&quot;. imho this is something that can wait for 4.7 because, while odd and perhaps even annoying, it doesn&#39;t result in any data loss. a simple &quot;Select one or more items from the list below&quot; string would probably help a lot. my other concern with putting this in without a string is that, with the bug fixed, we will forget to go in and add the string when we&#39;re open for string freeze. all imho.</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;">i see your point. I will try to   fix my page and if you validate i will commit when 4.7 will open.

Thanks for your review</pre>
<br />








<p>- Nicolas</p>


<br />
<p>On December 4th, 2010, 3 p.m., Nicolas Lécureuil wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://svn.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 Plasma.</div>
<div>By Nicolas Lécureuil.</div>


<p style="color: grey;"><i>Updated 2010-12-04 15:00:02</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;">When the user ask to show no buttons in the lockout widget, nothing tell the user that this is not possible, and when he saves the configuration nothing changed in the widget.

With this patch, if the user unselect all the checkboxes, he is warned.</pre>
  </td>
 </tr>
</table>




<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>


 <a href="https://bugs.kde.org/show_bug.cgi?id=256879">256879</a>


</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>trunk/KDE/kdebase/workspace/plasma/generic/applets/lock_logout/lockout.h <span style="color: grey">(1202613)</span></li>

 <li>trunk/KDE/kdebase/workspace/plasma/generic/applets/lock_logout/lockout.cpp <span style="color: grey">(1202613)</span></li>

</ul>

<p><a href="http://svn.reviewboard.kde.org/r/6041/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>








  </div>
 </body>
</html>