Review Request 111851: Split KCModules into different Categories in KHelpcenter's navigation tree

[Ben Cooksley]

> On Nov. 29, 2013, 7:32 p.m., Ben Cooksley wrote:
> > Patch looks fine to me, sucks a bit that the names have to be hardcoded for certain lists but not much can be done there if that is what the applications do...
> 
> Burkhard Lück wrote:
>     "Patch looks fine to me" = "Ship It!" ?

I was hoping someone else would chime in as well - but it can be shipped from my point of view, yes.


- Ben


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111851/#review44837
-----------------------------------------------------------


On Nov. 29, 2013, 12:37 p.m., Burkhard Lück wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111851/
> -----------------------------------------------------------
> 
> (Updated Nov. 29, 2013, 12:37 p.m.)
> 
> 
> Review request for Documentation, KDE Runtime, Albert Astals Cid, Ben Cooksley, David Faure, and Frank Reininghaus.
> 
> 
> Bugs: 262935
>     http://bugs.kde.org/show_bug.cgi?id=262935
> 
> 
> Repository: kde-runtime
> 
> 
> Description
> -------
> 
> 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. 
> 
> 
> Diffs
> -----
> 
>   khelpcenter/navigator.cpp 7460cc8 
>   khelpcenter/plugins/CMakeLists.txt d09b869 
>   khelpcenter/plugins/browsercontrolmodules.desktop PRE-CREATION 
>   khelpcenter/plugins/filemanagercontrolmodules.desktop PRE-CREATION 
>   khelpcenter/plugins/kcontrolmodules.desktop 1813df3 
>   khelpcenter/plugins/konquerorcontrolmodules.desktop PRE-CREATION 
>   khelpcenter/plugins/othercontrolmodules.desktop PRE-CREATION 
>   khelpcenter/plugintraverser.cpp b0b0e78 
> 
> Diff: http://git.reviewboard.kde.org/r/111851/diff/
> 
> 
> Testing
> -------
> 
> Checked with one example each for konquerorcontrol, browsercontrol, filemanagercontrol 
> and othercontrol, see attached screenshot.
> 
> 
> File Attachments
> ----------------
> 
> New Categories in KHelpcenter navigation tree
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/08/03/khelpcenter1.png
> 
> 
> Thanks,
> 
> Burkhard Lück
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20131213/f2d4fb84/attachment.htm>