----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111851/#review44080 -----------------------------------------------------------
kcmshell/main.cpp <http://git.reviewboard.kde.org/r/111851/#comment31595> 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. - Albert Astals Cid On Nov. 17, 2013, 9:53 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. 17, 2013, 9:53 p.m.) > > > Review request for Documentation, KDE Runtime and Albert Astals Cid. > > > 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 > >