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

(Updated Jan. 24, 2014, 9:08 a.m.)


Review request for kde-workspace and Aaron J. Seigo.


Changes
-------

Document use, avoid boolean trapping.


Repository: kde-workspace


Description
-------

Adds a second screen locking keyboard combination to provide advanced selective 
lock signaling to running applications for such things as pausing MPRIS2 
players in Kmix to setting away status in Telepathy. Applications are connected 
differently from standard screen saver signaling which is decidedly a blunt 
handled instrument.

A SecondaryActiveChanged signal for freedesktop.org in addition to 
ActiveChanged would be great to have moved into freedesktop.org with the above 
goal and cross-desktop in mind. Right now the implementation sits happily in 
org.kde.screensaver.


Diffs (updated)
-----

  ksmserver/screenlocker/dbus/org.kde.screensaver.xml e700b88 
  ksmserver/screenlocker/interface.h 97a6073 
  ksmserver/screenlocker/interface.cpp e3579fb 
  ksmserver/screenlocker/ksldapp.h b292b62 
  ksmserver/screenlocker/ksldapp.cpp 585cd5d 

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


Testing
-------

Compile, run. 


Thanks,

James Smith

Reply via email to