configChanged()

[Aaron J. Seigo]

On January 19, 2010, Marco Martin wrote:
> On Tue, Jan 19, 2010 at 8:46 AM, Aaron J. Seigo <aseigo at kde.org> wrote:
> > *sigh*
> > 
> > so i realized with a start this evening that virtually none of our
> > Plasmoids implement configChanged(). this means that the plasma-desktop
> > js scripting is going to be nearly useless with most of them in terms of
> > setting the configuration for them (without a plasma restart, of
> > course).
> > 
> > for applets that reimplement restore(KConfigGroup &config) it's probably
> > all good, though restore really shouldn't be reimplemented in the common
> > case.
> > 
> > it looks like one of our janitorial endeavors for 4.5 will be to
> > consolidate the configuration reading that goes on in plasmoids into
> > configChanged().
> 
> indeed we have to.
> unfortunately is a thing that is not "necessary" to make plasmoid
> work, so it will always be a pita to ensure plasmoid creators, even
> the new ones will respect it..

it's a little easier with scripted plasmoids, since that's how they tend to 
get notified that the config dialog has finished.

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Development Frameworks