<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="http://git.reviewboard.kde.org/r/102223/">http://git.reviewboard.kde.org/r/102223/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">As requested, the Yakuake side of things is now available for testing from a feature branch in yakuake.git, named feature-activity-and-silence-monitoring.
See the tab context menu to toggle monitoring. It's a single toggle action per type for tabs hosting a single terminal; for a tab hosting multiple (see the split options in the same context menu) it morphs into a sub-menu allowing toggling for all or individual terminals, highlighting the respective terminal as the menu actions are hovered.
The notification events can be configured from the usual "Configure Notifications" dialog, found from the Yakuake menu (the middle of the three buttons in the bottom-right corner of the window).
Note that much like Konsole, Yakuake takes care not to spam multiple activity notifications in short order. Only one notification for activity is shown, unless (a) monitoring is disabled and reenabled or (b) the active tab is switched or (c) the window is closed.
The Yakuake side is reasonably complete now, possibly pending some cleanup and version checks once the Konsole side is merged.</pre>
<br />
<p>- Eike</p>
<br />
<p>On August 5th, 2011, 1:35 p.m., Eike Hein wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Konsole.</div>
<div>By Eike Hein.</div>
<p style="color: grey;"><i>Updated Aug. 5, 2011, 1:35 p.m.</i></p>
<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;">The purpose of this patch is to allow applications hosting the Konsole KPart to tap into Konsole's ability to monitor terminals for silence or activity.
To accomplish this, Part has seen the addition of two slots, each taking a bool argument, to toggle the monitoring for silence or activity, respectively. Once and only if monitoring is enabled for an event type, a corresponding newly-added signal is emitted by Part.
Repeated calls to enable monitoring are handled by means of Qt::UniqueConnection. As the KPart internally has the notion of supporting multiple sessions, but this is not exposed in its API or user interface, the code has been written to toggle monitoring on all sessions by getting the list from SessionManager.
The use of signals and slots instead of extending the TerminalInterface class may seem strange, but this has ample precedent - there are several such slots and one signal already, which are in active use by applications (e.g. Yakuake).
The addition of this interface will allow the implementation of a long-standing feature wish by Yakuake users (monitoring for silence/activity).
Should this patch be accepted I can commit it myself.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=169650">169650</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/Part.h <span style="color: grey">(f1262dd)</span></li>
<li>src/Part.cpp <span style="color: grey">(e70b7e3)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102223/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>