<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/114567/">https://git.reviewboard.kde.org/r/114567/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for kde-workspace and Aaron J. Seigo.</div>
<div>By James Smith.</div>
<p style="color: grey;"><i>Updated Jan. 24, 2014, 9:08 a.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Document use, avoid boolean trapping.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kde-workspace
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Compile, run. </pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>ksmserver/screenlocker/dbus/org.kde.screensaver.xml <span style="color: grey">(e700b88)</span></li>
<li>ksmserver/screenlocker/interface.h <span style="color: grey">(97a6073)</span></li>
<li>ksmserver/screenlocker/interface.cpp <span style="color: grey">(e3579fb)</span></li>
<li>ksmserver/screenlocker/ksldapp.h <span style="color: grey">(b292b62)</span></li>
<li>ksmserver/screenlocker/ksldapp.cpp <span style="color: grey">(585cd5d)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/114567/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>