configChanged()

Marco Martin notmart at gmail.com
Wed Jan 20 06:18:51 CET 2010


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..
 Marco MArtin


More information about the Plasma-devel mailing list