----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105319/#review15062 -----------------------------------------------------------
Ship it! Looks great. It's a new feature though so don't ship it till after the feature freeze ends for 4.9. (unless someone else gives you permission) - David Edmundson On June 24, 2012, 3:47 p.m., Varun Herale wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105319/ > ----------------------------------------------------------- > > (Updated June 24, 2012, 3:47 p.m.) > > > Review request for Plasma. > > > Description > ------- > > This patch is for hosting a dbus-interface that can be used to load any > installed wallpaper plugin onto current desktop containment. In case of > default "image" plugin, the path to the image can also be sent which will > change the wallpaper. > > > Diffs > ----- > > plasma/desktop/shell/dbus/org.kde.plasma.App.xml eefce32 > plasma/desktop/shell/plasmaapp.h 6ae0c89 > plasma/desktop/shell/plasmaapp.cpp 7abd8fc > > Diff: http://git.reviewboard.kde.org/r/105319/diff/ > > > Testing > ------- > > Tested on different activities and made sure it works for per-virtual desktop > containment. > > Haven't tested on a system with multiple screens though, as I don't have > access to one. Could someone please test for that ? > > > Thanks, > > Varun Herale > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
