preview in wallpaper dialog

[Aaron J. Seigo]

On Thursday, August 12, 2010, Marco Martin wrote:
> recapping:

good summary.

> in the end, if a little live preview is needed should be up
 to the
> wallpaper implelentation.

yes.

> the attached patch is a first quick and dirty implementation of
 tat:
> the need of the monitor is decided by Wallpaper::setNeedsPreview(bool)
> that at the moment is default off

may setPreviewDuringConfiguration? yes, long, but also very clear :)

>
 Image would have it off, slideshow on, mandelbrot and barble on and
> whatever each individual wallpaper needs it or not.

again: why does slideshow need it? it just shows a random image from the mix
 you add. i don't see how it is actually useful?
 
> the monitor is now put uder the plugin selection combobox giving a
 vertical
> layout.

cool; i think this should go in... 

as an asid: i also think that the
pattern plugin should not have the preview and should instead show an
 iconview containing the various patterns.

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

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