Review Request: Plasma D-Bus Interface: Setting wallpaper image

Ivan Cukic ivan.cukic+kde at gmail.com
Tue Oct 6 13:28:01 CEST 2009


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

Review request for Plasma.


Summary
-------

There are many users who want the way to set the wallpaper via d-bus.

This enables them to do so, but only for the image wallpaper.

Since there is no mechanism in Plasma::Wallpaper (as far as I know) to set wallpaper options from outside of the wallpaper plugin, this patch relies on the structure of the Image wallpaper plugin and its configuration file format.


Diffs
-----

  /trunk/KDE/kdebase/workspace/plasma/desktop/shell/dbus/org.kde.plasma.App.xml 1031712 
  /trunk/KDE/kdebase/workspace/plasma/desktop/shell/plasmaapp.h 1031712 
  /trunk/KDE/kdebase/workspace/plasma/desktop/shell/plasmaapp.cpp 1031712 

Diff: http://reviewboard.kde.org/r/1798/diff


Testing
-------

Testing done - changing the wallpaper from one image to another, from another plugin to image plugin.


Thanks,

Ivan



More information about the Plasma-devel mailing list