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





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On May 12th, 2010, 1:33 p.m., <b>Oswald Buddenhagen</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;">> But on the other hand I can suggest probably a better place for this - kdm itself ;)
>
no, it may very well make sense to accept way more than what should be permitted via the kcm. in fact, my own local setup is totally insecure for practical reasons.

i'll polish up the coding style of your patch and commit it in the next few days. thanks for the effort. you survived it. :)</pre>
 </blockquote>




 <p>On May 14th, 2010, 3:51 p.m., <b>Igor Krivenko</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'll polish up the coding style of your patch and commit it in the next few days.
Thank you very much. I hope this patch will stimulate popularization and development of KAuth :)</pre>
 </blockquote>





 <p>On March 2nd, 2011, 2:44 p.m., <b>Cyberbeat</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;">was this patch committed?</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;">months ago. it should be marked as such ...</pre>
<br />








<p>- Oswald</p>


<br />
<p>On May 12th, 2010, 11:16 a.m., Igor Krivenko 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 kdelibs, usability, Dario Freddi, Oswald Buddenhagen, and Frederik Gladhorn.</div>
<div>By Igor Krivenko.</div>


<p style="color: grey;"><i>Updated May 12, 2010, 11:16 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;">This is a patch for KDM kcontrol module.
It implements Kauth support for changing kdm settings and managing user images. 
It still doesn't support installation/deletion of themes (including installation via knewstuff).
As far as I know there were plans to port KNewStuff3 to KAuth, so a more consistent approach is
to rely on the KNewStuff's integration when it's available.
</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>/trunk/KDE/kdebase/workspace/kcontrol/kdm/CMakeLists.txt <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/background.h <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/background.cpp <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/helper.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/helper.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/kcmkdm_actions.actions <span style="color: grey">(PRE-CREATION)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/kdm-conv.h <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/kdm-conv.cpp <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/kdm-dlg.h <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/kdm-dlg.cpp <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/kdm-gen.h <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/kdm-gen.cpp <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/kdm-shut.h <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/kdm-shut.cpp <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/kdm-theme.h <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/kdm-theme.cpp <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/kdm-users.h <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/kdm-users.cpp <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/main.h <span style="color: grey">(1125102)</span></li>

 <li>/trunk/KDE/kdebase/workspace/kcontrol/kdm/main.cpp <span style="color: grey">(1125102)</span></li>

</ul>

<p><a href="http://svn.reviewboard.kde.org/r/3631/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://svn.reviewboard.kde.org/r/3631/s/360/"><img src="http://svn.reviewboard.kde.org/media/uploaded/images/2010/04/17/snapshot1_400x100.png" style="border: 1px black solid;" alt="Saving KDM settings" /></a>

 <a href="http://svn.reviewboard.kde.org/r/3631/s/361/"><img src="http://svn.reviewboard.kde.org/media/uploaded/images/2010/04/17/snapshot2_400x100.png" style="border: 1px black solid;" alt="Managing user images" /></a>

</div>


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








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