On Saturday 23 August 2008, Petri Damstén wrote:
> Containment calls restore for wallpaper like this d->wallpaper-
>
> >restore(KConfigGroup(&cfg, "Wallpaper"), mode); and wallpaper plugin tries
> > to
>
> read it's config values from that but there is none (I think) since this is
> the config group from the new containment?

ah .. it calls restore on the paper inside of setWallpaper. ugh.

so ... a new approach is called for as we're going to run into this same issue 
(nested config groups) in a number of places. i think we should probably just 
do something like this in swapContainment (completely untested =):

KConfigGroup oldConfig = old->config();
oldConfig->copyTo(c->config());

this *should* bring over all the configurations: applets, wallpapers, 
everything. then we can call restore on the new containment with it's own 
config object ..

the only problem here would be buggy applets that don't save their state when 
asked to, but then they'd be buggy to the point of not working on shutdown 
either so this shouldn't matter.

-- 
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 Trolltech

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to