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

[Burkhard Lück]

> On Nov. 20, 2013, 11:50 p.m., Albert Astals Cid wrote:
> > kcmshell/main.cpp, line 63
> > <http://git.reviewboard.kde.org/r/111851/diff/1/?file=176023#file176023line63>
> >
> >     I'm not sure changing the possible values of X-KDE-ParentApp is a good idea at this stage, it may break unexpectedly in other places.
> >     
> >     If there is no other way to detect the "browsercontrol" files, i'd suggest creating another X-KDE-Something key instead of changing the existing one.

http://api.kde.org/4.x-api/kdelibs-apidocs/kutils/html/namespaceKSettings.html:

"X-KDE-ParentApp is the name of the application this config page belongs to"

So this seems to be the right desktop item to use.

grepping the code of all kde main modules in 4.12 for all occurrences like "if X-KDE-ParentApp == 'kcontrol' shows only two hits:

kde-runtime/khelpcenter/navigator.cpp:228
kde-runtime/kcmshell/main.cpp:63

And both hits are handled in this review.


- Burkhard


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


On Nov. 21, 2013, 11:50 a.m., Burkhard Lück wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111851/
> -----------------------------------------------------------
> 
> (Updated Nov. 21, 2013, 11:50 a.m.)
> 
> 
> Review request for Documentation, KDE Runtime, Albert Astals Cid, Ben Cooksley, and David Faure.
> 
> 
> 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/20131121/691e3aa6/attachment.htm>
-------------- next part --------------
_______________________________________________
kde-doc-english mailing list
kde-doc-english at kde.org
https://mail.kde.org/mailman/listinfo/kde-doc-english