QML-only KCM base
Review Request #122886 - Created March 10, 2015 and submitted
This is a QObject-based fork of KCmodule, with the api slightly adaped to be more declarative friendly.
it loads a qml file that then will be able to access its instance as "kcm" or as the "KCM" attached property, like what happens in plasmoids (for how attached proeprties work, unfortunately only properties of the base class will be accessible).
They will be loaded in systemsettings by review 122887
|I don't particularly like the KDeclarative namespace here since the library is called QuickAddons, but unfortunately that name is terrible, ...||Marco Martin|
|I didn't test things that need authentication yet (if some good soul ports a module that uses it would be ...||Marco Martin|
|this should mention packages somewhere.||David Edmundson|
|This documentation tells me nothing, and I'm not sure what this does from the method name.||David Edmundson|
|I guess you copy and pasted all these. KConfigXT isn't relevant at all here.||David Edmundson|
|when combined with your other KCModule patch you have a crash. From KCModule docs: This sets the KAboutData returned by ...||David Edmundson|
|you need also KF5::Auth as public, the new public header has a kauthaction include||Hrvoje Senjan|
this should mention packages somewhere.
This documentation tells me nothing, and I'm not sure what this does from the method name.
I guess you copy and pasted all these.
KConfigXT isn't relevant at all here.
when combined with your other KCModule patch you have a crash.
From KCModule docs:
This sets the KAboutData returned by aboutData() The about data is now owned by KCModule.
So this will result in a double delete.
Same for the delete in setAboutData.
In this version, at least the namespace is KQuickAddons
also, the old include QuickAddons/ is still there just for compatibility, new stuff is intended to go in KQuickAddons/
That's still ugly, but I think at least makes things a bit better for future additions (even regardless ConfigModule ends there, which i prefer, or in KDeclarative)
Revision 5 (+786 -1)