> On 2010-04-06 14:31:17, Marco Martin wrote: > > uhm, is it enough of an use case for the feature? > > or wouldn't the proper solution to make the wallpaper plugin to rotate the > > pictures accordingly to the exif tag? > > Jeremy Whiting wrote: > Hmm, others (maybe even me once I learn how) might also want to use it to > get rid of redeye, crop images, and other stuff too. That was just my > immediate use case is all. What do you think?
yes, makes sense - Marco ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3498/#review4892 ----------------------------------------------------------- On 2010-04-06 02:59:03, Jeremy Whiting wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/3498/ > ----------------------------------------------------------- > > (Updated 2010-04-06 02:59:03) > > > Review request for Plasma, Aaron Seigo and Chani Armitage. > > > Summary > ------- > > I use slideshow mode for my wallpaper and occasionally an image appears that > needs to be rotated, but I don't know where the file is. I added a context > action to slideshow wallpaper that will open the current image in the image > app the user has configured (gwenview by default I believe) so it can then be > rotated/fixed whatever. > > > Diffs > ----- > > trunk/KDE/kdebase/workspace/plasma/generic/wallpapers/image/image.h 1109045 > trunk/KDE/kdebase/workspace/plasma/generic/wallpapers/image/image.cpp > 1109045 > > Diff: http://reviewboard.kde.org/r/3498/diff > > > Testing > ------- > > I tested it on my machine that has trunk built and it seems to work fine. > > > Thanks, > > Jeremy > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
