Review Request: add "Open Image" action to slideshow wallpaper context menu

[Marco Martin]

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