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

[David Faure]

> On Nov. 28, 2013, 8:42 p.m., Burkhard Lück wrote:
> > In theory I have a possible solution using the existing X-KDE-PluginKeyword in the desktop files, 
> > so no changes outside khelpcenter would be required.
> > 
> > But in contrary to the api documentation this does not work or I do wrong:
> > 
> > $ ktraderclient --servicetype KCModule --constraint "[X-KDE-PluginKeyword] == 
> > 'dolphinviewmodes'"
> > servicetype is : KCModule
> > constraint is : [X-KDE-PluginKeyword] == 'dolphinviewmodes'
> > got 0 offers.
> > 
> > This should return *one* offer kcmdolphinviewmodes.desktop which has
> > X-KDE-ServiceTypes=KCModule
> > +
> > X-KDE-PluginKeyword=dolphinviewmodes
> > 
> > What is wrong here?
> >

This is what's wrong:

KServicePrivate::property: Request for unknown property ' "X-KDE-PluginKeyword" '

Fixed by defining this property in kdelibs/kio/kcmodule.desktop:

--- a/kio/kcmodule.desktop
+++ b/kio/kcmodule.desktop
@@ -114,3 +114,8 @@ Type=QString
 # sets the order of the modules in the TreeList/IconList
 [PropertyDef::X-KDE-Weight]
 Type=int
+
+# TODO: document
+[PropertyDef::X-KDE-PluginKeyword]
+Type=QString


- David


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


On Nov. 28, 2013, 8:36 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. 28, 2013, 8:36 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
> -----
> 
>   kcmshell/main.cpp dab8fc6 
>   khelpcenter/navigator.cpp 7460cc8 
>   khelpcenter/plugins/CMakeLists.txt d09b869 
>   khelpcenter/plugins/browsercontrolmodules.desktop e69de29 
>   khelpcenter/plugins/filemanagercontrolmodules.desktop e69de29 
>   khelpcenter/plugins/kcontrolmodules.desktop 1813df3 
>   khelpcenter/plugins/konquerorcontrolmodules.desktop e69de29 
>   khelpcenter/plugins/othercontrolmodules.desktop e69de29 
>   khelpcenter/plugintraverser.cpp b0b0e78 
>   kioslave/cgi/kcmcgi/kcmcgi.desktop f49eeb9 
> 
> 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/20131128/cfb13038/attachment.htm>