Wallpaper plugins

[Petri Damstén]

On Saturday 23 August 2008 17:57:53 Aaron J. Seigo wrote:
> On Saturday 23 August 2008, Petri Damstén wrote:
> > If I have understood correctly restore calls setWallpaper which gives
> > containments own config (not the old containments config used in restore)
> > to wallpaper plugins. So should I fix setWallpaper so it can take the
> > KConfigGroup from restore?
>
> it compares the plugin and mode against what is in the containment's own
> config, but that's ok in this case because:
>
> * in restore, the values are read from the old containment's config
>
> * these values are then passed to setWallpaper
>
> * setWallpaper compares those values to the current wallpaper plugin (if
> any), and changes the wallpaper plugin if the values aren't the same

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?

> * setWallpaper then saves the configuration to the current containment's
> config
>
> so from what i'm reading in the code, it should work ...

Petri

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20080823/950f362c/attachment.sig