Review Request: Adding "Set Wallpaper Image" Option in Picture Frame

Sinny Kumari ksinny at gmail.com
Wed Jan 26 17:20:08 CET 2011


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

Review request for Plasma.


Summary
-------

Including this patch and other two (http://svn.reviewboard.kde.org/r/6375/ , http://reviewboard.kde.org/r/6391/ ) , It adds "Set Wallpaper Image" feature in Picture Frame. If the User right clicks on Picture Frame, there will be an option "Set Wallpaper Image". This Option will set the current Image Of Picture
Frame as Wallpaper Image.


Diffs
-----

  trunk/KDE/kdeplasma-addons/applets/frame/frame.h 1216449 
  trunk/KDE/kdeplasma-addons/applets/frame/frame.cpp 1216449 

Diff: http://svn.reviewboard.kde.org/r/6416/diff


Testing
-------

Setting Picture Frame Image as wallpaper Image in all cases. Added QTimer in method Frame::setImageAsWallpaper() in Order to work properly in case of Wallpaper other than Image and Slideshow. If there is better solution than using QTimer, please suggest :)


Thanks,

Sinny

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20110126/e9b295d2/attachment.htm 


More information about the Plasma-devel mailing list