https://bugs.kde.org/show_bug.cgi?id=481113

--- Comment #59 from David Edmundson <k...@davidedmundson.co.uk> ---
Edit: I *think* the old code was meant to be:

 - on change start a applet-wide timer and save to the kconfig instance
 - that emits a signal that tells us to sync, which starts a global timer to do
a  sync to disk

I removed the former with e08eb3caf329df648d39de8db5182bb862569ef5 as it was
clearly broken and not hooked up to anything, but I think the problem was that
it should have been.

I'm I'm right I should revert that, and change:

+        connect(d->configPropertyMap, &KConfigPropertyMap::valueChanged, this,
&Applet::configNeedsSaving);

to call 

connect(d->configPropertyMap, &KConfigPropertyMap::valueChanged, this,
[this](){ d->scheduleModificationNotification()});

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to