<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/118796/">https://git.reviewboard.kde.org/r/118796/</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;">This review has been submitted with commit 299ba613208897be49383f41a2c30016f56fe31a by Martin Klapetek to branch master.</pre>
<br />
<p>- Commit Hook</p>
<br />
<p>On June 20th, 2014, 12:21 p.m. UTC, Martin Klapetek wrote:</p>
<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 Plasma.</div>
<div>By Martin Klapetek.</div>
<p style="color: grey;"><i>Updated June 20, 2014, 12:21 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
plasma-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;">This prevents some apps to spam lots of notifications when all it needs is actually just one single notification.
Imagine you're switching songs in your playlist quickly and each song change sends new notification, but when
you get to the tenth song, you still see the notification from the 3rd song (because timeouts) and you don't
really care about all those songs changes still in the queue as you see it in the playlist anyway.
So this patch limits certain apps to have only one single notification which is always updated. So far
Clementine and Spotify is there. Switching songs or changing playback status creates only one single popup.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>applets/notifications/package/contents/ui/NotificationPopup.qml <span style="color: grey">(d07ee32)</span></li>
<li>applets/notifications/plugin/notificationshelper.cpp <span style="color: grey">(c5a1082)</span></li>
<li>dataengines/notifications/notificationsengine.h <span style="color: grey">(df0d3ca)</span></li>
<li>dataengines/notifications/notificationsengine.cpp <span style="color: grey">(aaf6079)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/118796/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>