<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/109825/">http://git.reviewboard.kde.org/r/109825/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://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 KDEPIM and Plasma.</div>
<div>By Martin Klapetek.</div>
<p style="color: grey;"><i>Updated April 10, 2013, 4:09 p.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;">Changes:
* moved collection remote ids to the header file
* added "Mark all notifications as read" option to KSNI
* added "Hide" option to KSNI
* made the KSNI tooltip richtext and used <ul> for the items (makes them better visually separated)
* limited KNotification to display only 1 notification as it is forced to always have 4 lines, so more notifications wouldn't be displayed anyway
* reworked the logic of the string creations a bit
* fixed issues found by Kevin</pre>
</td>
</tr>
</table>
<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;">This patch displays a KNotification whenever an unread notification exists on Facebook. This KNotification groups at most 3 notifications into one popup and then say "...and N more" if there is more. It also keeps track of which notifications were already displayed and does not display them again unless they were updated on the server. This is all stored in a separate config file.
Then it creates a KSNI for the notifications where it always show the newest three notifications in the tooltip (regardless if it was displayed before or not) and creates a menu with the notifications, which opens browser with the notification link.</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;">Yes.</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>resources/facebook/CMakeLists.txt <span style="color: grey">(e8c6381)</span></li>
<li>resources/facebook/facebookresource.h <span style="color: grey">(4a16c0c)</span></li>
<li>resources/facebook/facebookresource.cpp <span style="color: grey">(67e8f3b)</span></li>
<li>resources/facebook/facebookresource_events.cpp <span style="color: grey">(9fc8b31)</span></li>
<li>resources/facebook/facebookresource_notifications.cpp <span style="color: grey">(7f6b8c4)</span></li>
<li>resources/facebook/serializer/akonadi_serializer_socialnotification.cpp <span style="color: grey">(a261e14)</span></li>
<li>resources/facebook/settingsbase.kcfg <span style="color: grey">(9f8e4b5)</span></li>
<li>resources/facebook/settingsdialog.cpp <span style="color: grey">(bfb7826)</span></li>
<li>resources/facebook/settingsdialog.ui <span style="color: grey">(68b6a24)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/109825/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<ul>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/04/02/facebook_notifications.png">KSNI</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>