Review Request 128950: Allow Plasma desktop scripting to change panel's screen.
Kamil Sołtysik
kamil at soltysik.in
Tue Sep 20 14:25:00 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128950/
-----------------------------------------------------------
(Updated Sept. 20, 2016, 2:25 p.m.)
Review request for Plasma.
Changes
-------
Fixed problems noticed by reviewers.
Bugs: https://bugs.kde.org/show_bug.cgi?id=363592
https://bugs.kde.org/show_bug.cgi?id=https://bugs.kde.org/show_bug.cgi?id=363592
Repository: plasma-workspace
Description
-------
As stated in doc here: https://userbase.kde.org/KDE_System_Administration/PlasmaDesktopScripting#Panels Panels should allow to be placed on selected screen by setting value of Panel::screen property. While looking at code I found out that this property is read-only in code. Patch changes that.
Diffs
-----
shell/scripting/panel.h 0d70784
shell/scripting/panel.cpp b9383df
Diff: https://git.reviewboard.kde.org/r/128950/diff/
Testing
-------
Panel's position can now be changed by setting the property. Setting invalid value causes no response.
File Attachments (updated)
----------------
plasma-workspace-2.patch
https://git.reviewboard.kde.org/media/uploaded/files/2016/09/20/bc1df578-8758-48e4-b2b2-a53792c803f8__plasma-workspace-2.patch
Thanks,
Kamil Sołtysik
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160920/2145ab5e/attachment.html>
More information about the Plasma-devel
mailing list