preview in wallpaper dialog

[Marco Martin]

On Thursday 12 August 2010, Aaron J. Seigo wrote:
> 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 :)

yes, indeed, would still prefix the getter with needs tough?

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

maybe if the image switches every 2-3 seconds could give an overview of what 
those folders contain.
not strictly necessary, bbut somewhat remotely 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.
i have a local patch that partly implements it already :p

now the thing is:
some of the pattern images make sense painted a single time, some are very 
timy and need to be tiled several times to make sense.
so, the question is: usual problem of not being able to use Wallpaper itself 
to generate thumbnails and the tile generation have to be duplicated in the 
code just for the thumbnails?

Cheers,
Marco Martin