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





 <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 don't see this being important enough to warrant new strings and new code into 4.11 at this very late stage</pre>
 <br />









<p>- Albert</p>


<br />
<p>On August 3rd, 2013, 11:21 a.m. UTC, Burkhard Lück wrote:</p>








<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 Documentation and KDE Runtime.</div>
<div>By Burkhard Lück.</div>


<p style="color: grey;"><i>Updated Aug. 3, 2013, 11:21 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;">The KHelpcenter navigation tree has a top level item "Control Center Modules", 
an unsorted list of all KCModules (>80 for a full kde main modules install from stable).
This makes this item hardly usable, see https://bugs.kde.org/show_bug.cgi?id=262935

This patch implements:

A) Alphabetical sorting for "Control Center Modules"/"Foo Settings Modules" + "KInfoCenter" items

B) New / changed top level categories in the navigation tree:

   1) System Settings Modules -> replaces old "Control Center Modules"
      items: see http://docs.kde.org/stable/en/kde-workspace/systemsettings/general.html

   2) Konqueror Settings Modules (see Konqueror settings dialog)
      items: General, Performance, Bookmarks 

   3) Filemanager Settings Modules (see Konqueror/Dolphin settings dialog)
      items:
      File Management, View Modes, Navigation, Services, General, Trash
      No File Associations, because it is already in System Settings Modules

   4) Browser Settings Modules (see Konqueror settings dialog)
      items:
      Web Browsing, Proxy, Appearance, AdBlocK Filters, Web Shortcuts, Cache, 
      History, Cookies, Browser Identification, Java & JavaScript, Plugins

   5) Other Settings Modules
      all other items like e.g. CGI Scripts from kde-runtime

   To make full use of these new/changed categories some kcm desktop files in other 
   modules than kde-runtime need a change of X-KDE-ParentApp from kcontrol to konquerorcontrol,
   browsercontrol, filemanagercontrol or othercontrol, but that is not part of this review.
   As long as not all desktop files are fixed according to this patch or a necessary change in a 
   desktop files is overlooked that KCM will be in System Settings Modules like now, but in sorted order.

C) Change wording from "Control Center Modules" to "System Settings Modules" and using 
   "Foo Settings Modules" for the new categories.
   Control Center is from KDE 3, we use System Settings nearly all over GUI and in the whole documentation. 
</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;">Checked with one example each for konquerorcontrol, browsercontrol, filemanagercontrol 
and othercontrol, see attached screenshot.</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="http://bugs.kde.org/show_bug.cgi?id=262935">262935</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>kcmshell/main.cpp <span style="color: grey">(dab8fc6)</span></li>

 <li>khelpcenter/navigator.cpp <span style="color: grey">(7460cc8)</span></li>

 <li>khelpcenter/plugins/CMakeLists.txt <span style="color: grey">(d09b869)</span></li>

 <li>khelpcenter/plugins/browsercontrolmodules.desktop <span style="color: grey">(e69de29)</span></li>

 <li>khelpcenter/plugins/filemanagercontrolmodules.desktop <span style="color: grey">(e69de29)</span></li>

 <li>khelpcenter/plugins/kcontrolmodules.desktop <span style="color: grey">(1813df3)</span></li>

 <li>khelpcenter/plugins/konquerorcontrolmodules.desktop <span style="color: grey">(e69de29)</span></li>

 <li>khelpcenter/plugins/othercontrolmodules.desktop <span style="color: grey">(e69de29)</span></li>

 <li>khelpcenter/plugintraverser.cpp <span style="color: grey">(b0b0e78)</span></li>

 <li>kioslave/cgi/kcmcgi/kcmcgi.desktop <span style="color: grey">(f49eeb9)</span></li>

</ul>

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



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>

<ul>

 <li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/08/03/khelpcenter1.png">New Categories in KHelpcenter navigation tree</a></li>

</ul>





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








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