Review Request: Suppress Kickoff Computer tab entries from appearing in Applications tab more cleverly

[Will Stephenson]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://mattr.info:8080/r/360/
-----------------------------------------------------------

Review request for Plasma.


Summary
-------

in the kde-core-devel thread "Kubuntu Settings in KDE" Riddell suggested setting NoDisplay=true on systemsettings.desktop, to prevent it appearing twice in the Computer tab, and in the Applications tab.  This solution goes too far, because you then cannot find the app with the search tool.

This patch adapts the Applications tab to ignore any apps that are configured to appear in the System tab, using a new static method in the Kickoff namespace to obtain the list in both System and Applications models.


Diffs
-----

  trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/applicationmodel.cpp
  trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/models.h
  trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/models.cpp
  trunk/KDE/kdebase/workspace/plasma/applets/kickoff/core/systemmodel.cpp

Diff: http://mattr.info:8080/r/360/diff


Testing
-------

The usual compile it and check what it does testing.


Thanks,

Will