Follow-up from "Port KCM loading to KPluginMetaData"
The following discussions from !76 (merged) should be addressed:
-
@sitter started a discussion: (+1 comment) I would really like @mart to take a look at this. It's not exactly hot. Shouldn't/didn't weight take care of this?
-
@sitter started a discussion: (+1 comment) Couldn't/Shouldn't the loader function only return not hidden metadata? Seems a bit silly to even have to deal with this outside the factorization function. Hidden stuff may never be shown anyway, doesn't it?
-
@sitter started a discussion: (+1 comment) The external app stuff doesn't seem to be working. KInfocenter+plasma-systemmonitor from master should give you a working example. KIC should have a monitor entry but it doesn't for me.
-
@sitter started a discussion: It irks me greatly that this is like the fourth time I am reading the the name of the landingpage spelled out. If I was you I would put it in a static function in the menuitem header
static QString settingsLandingPluginId()
and use that in place of all the hardcoding. Same forinfoLandingPluginId()
so we have balance in the force.