The category was only used for proper Plasma::Applet instances but not unloaded plasmoids from pure metadata.
BUG: 416947
FIXED-IN: 5.18.0
kmaterka |
Plasma |
The category was only used for proper Plasma::Applet instances but not unloaded plasmoids from pure metadata.
BUG: 416947
FIXED-IN: 5.18.0
Automatic diff as part of commit; lint not applicable. |
Automatic diff as part of commit; unit tests not applicable. |
I definitely need to split D26992 into smaller chunks :) I had it fixed, but obviously D26992 can't be merged for 5.18.
There is a second inconsistency, but it is not a serious problem. There are some category overrides in main.qml, if you think it is a problem I will prepare separate revision with something like this:
static const QHash<QString, QString> s_categoryOverride = { {QStringLiteral("org.kde.discovernotifier"), QStringLiteral("SystemServices")}, {QStringLiteral("org.kde.plasma.networkmanagement"), QStringLiteral("Hardware")}, {QStringLiteral("org.kde.kdeconnect"), QStringLiteral("Hardware")}, {QStringLiteral("org.kde.plasma.keyboardindicator"), QStringLiteral("Hardware")}, {QStringLiteral("touchpad"), QStringLiteral("Hardware")} };
category overrides in main.qml
I think those should go away and have the SNI/plasmoid in the right category. I think most of them are obsolete already anyway, e.g. org.kde.plasma.networkmanagement is Hardware in its metadata