Review Request 118038: Bring back a KCM for the screen locker

Martin Gräßlin mgraesslin at kde.org
Thu May 8 05:16:40 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118038/
-----------------------------------------------------------

(Updated May 8, 2014, 7:16 a.m.)


Review request for Plasma and Thomas Pfeiffer.


Changes
-------

Incorporated suggested changes except the comboboxes


Repository: plasma-workspace


Description
-------

Bring back a KCM for the screen locker

Note: I will improve the default values to get them sane. That needs adjustements in the screenlocker source base and I want to get the review of the UI started earlier.


Diffs (updated)
-----

  ksmserver/screenlocker/kcm/screenlocker.desktop PRE-CREATION 
  ksmserver/screenlocker/kcm/kcm.ui PRE-CREATION 
  ksmserver/screenlocker/kcm/CMakeLists.txt PRE-CREATION 
  ksmserver/screenlocker/kcm/kcm.cpp PRE-CREATION 
  ksmserver/screenlocker/CMakeLists.txt 3407ea46d5046f2f5b2e7321d714ad1e65ffceea 

Diff: https://git.reviewboard.kde.org/r/118038/diff/


Testing
-------


File Attachments
----------------

KCM with values
  https://git.reviewboard.kde.org/media/uploaded/files/2014/05/07/f70f34d6-2c5c-4638-8e43-d9d727fdb7cb__kcm-with-values.png
KCM without values
  https://git.reviewboard.kde.org/media/uploaded/files/2014/05/07/89b1ef8e-803f-42fb-b2cb-52b776246598__kcm-without-values.png


Thanks,

Martin Gräßlin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140508/3af9a677/attachment.html>


More information about the Plasma-devel mailing list