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

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