Commit b9a54826 authored by David Edmundson's avatar David Edmundson
Browse files

[kcms/launchfeedback] Port to KConfig notifiers

Summary:
This gets rid of all C++ code in this KCM!

It gets rid of a kwin tie in where one shouldn't exist and also exposes
a notification that the taskmanager should be able to hook into -
something currently missing as it loads this setting.

Test Plan: Changed setting, kwin updates automatically

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D25835
parent 8b00bcb9
......@@ -53,14 +53,4 @@ LaunchFeedbackSettings *LaunchFeedback::launchFeedbackSettings() const
return m_settings;
}
void LaunchFeedback::save()
{
ManagedConfigModule::save();
org::kde::kwin::Effects kwin(QStringLiteral("org.kde.KWin"),
QStringLiteral("/Effects"),
QDBusConnection::sessionBus());
kwin.reconfigureEffect(QStringLiteral("startupfeedback"));
}
#include "launchfeedback.moc"
......@@ -44,9 +44,6 @@ public:
LaunchFeedbackSettings *launchFeedbackSettings() const;
public Q_SLOTS:
void save() override;
private:
LaunchFeedbackSettings *m_settings;
};
......
......@@ -4,3 +4,4 @@ Mutators=true
DefaultValueGetters=true
GenerateProperties=true
ParentInConstructor=true
Notifiers=true
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment