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

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?


- Jeremy


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

Reply via email to