> On Dec. 21, 2012, 12:28 a.m., Kai Uwe Broulik wrote: > > plasma/generic/wallpapers/image/image.cpp, line 589 > > <http://git.reviewboard.kde.org/r/107821/diff/1/?file=100482#file100482line589> > > > > I give you an “i” ;) > > Erwan MATHIEU wrote: > What do you mean ? > > Jeremy Paul Whiting wrote: > He means you probably want to use "image" rather than "mage" as they are > quite different things afterall.
Oh yes, you're damn right !! - Erwan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107821/#review23787 ----------------------------------------------------------- On Dec. 20, 2012, 8:14 p.m., Erwan MATHIEU wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/107821/ > ----------------------------------------------------------- > > (Updated Dec. 20, 2012, 8:14 p.m.) > > > Review request for kde-workspace. > > > Description > ------- > > Patch which corrects bug 235710 : Plasma Wallpaper Slideshow to periodially > recheck contents of image folder > > > This addresses bug 235710. > http://bugs.kde.org/show_bug.cgi?id=235710 > > > Diffs > ----- > > plasma/generic/wallpapers/image/backgroundlistmodel.h a289f49 > plasma/generic/wallpapers/image/backgroundlistmodel.cpp f5a4dce > plasma/generic/wallpapers/image/image.h 417f5a7 > plasma/generic/wallpapers/image/image.cpp 006a748 > > Diff: http://git.reviewboard.kde.org/r/107821/diff/ > > > Testing > ------- > > Many pictures in many folders with many subfolders, adding and removing files > (sometimes no more files available) > > > Thanks, > > Erwan MATHIEU > >
