[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: preview in wallpaper dialog
From: Marco Martin <notmart () gmail ! com>
Date: 2010-08-12 17:13:46
Message-ID: 201008121913.46720.notmart () gmail ! com
[Download RAW message or body]
On Thursday 12 August 2010, Aaron J. Seigo wrote:
> On Thursday, August 12, 2010, Marco Martin wrote:
> > 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?
>
> true...
okie, so the first version will go in now, then up for next modifications :)
> > > 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.
>
> personally i'd prefer an icon view showing all entries ... that would be
> actually useful as you could see all the images in the collection. with
> delayed "only render what is visible" like we have already in the
> SingleImage preview case it hopefully should not be a problem.
hmm, folders could be huge, i think it could throw in too much information
> > 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?
>
> it's a one-liner in QPainter to tile a pixmap, so i don't think it's a big
> deal :)
yup, in this case indeed. was thinking also on a more general point tough
--
Marco Martin
_______________________________________________
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]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic