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

Marco Martin notmart at gmail.com
Tue Apr 6 16:31:13 CEST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3498/#review4892
-----------------------------------------------------------


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?

- Marco


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



More information about the Plasma-devel mailing list