Review Request 114567: RFC: SecondaryActiveChanged signal for advanced screenlocking (global pause MPRIS2, Telepathy N/A status, etc)

[David Edmundson]

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


Can I confirm that I've understood this patch correctly.
The goal is to have a second keyboard that locks the screen. Applications that monitor activeChanged will be able to tell if the screen was locked specifically because this second shortcut was pressed as opposed to any other lock cause.


- David Edmundson


On Jan. 24, 2014, 9:08 a.m., James Smith wrote:
> 
> -----------------------------------------------------------
> 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.
> 
> 
> 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
> -----
> 
>   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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20140208/4e82c291/attachment.htm>