[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: Wallpaper plugins
From:       Petri =?iso-8859-15?q?Damst=E9n?= <petri.damsten () gmail ! com>
Date:       2008-08-23 15:09:09
Message-ID: 200808231809.14884.petri.damsten () gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


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


["signature.asc" (application/pgp-signature)]

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


[prev in list] [next in list] [prev in thread] [next in thread]