<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/110389/">http://git.reviewboard.kde.org/r/110389/</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 Plasma and KDE Usability.</div>
<div>By Kai Uwe Broulik.</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 patch adds a configure button to notifications that, when clicked, open the KNotify settings for the app that emitted the notification.
This makes it much easier to turn off a notification you do not want and you have no clue where to turn it off (no novice user would know that you can actually turn these off, not even speaking of where)
I asked Olivier Goffart to use his notifyrc lookup code that is licensed GPLv2 in this LGPLv2 code. I am not quite happy with it, though. Iterating over all notifyrc files, hoping to find a match, isn't exactly a good thing. I have no idea how I could do it in a different way, though. According to the specs the application name passed to the notification system is "[…] the optional name of the application sending the notification. This should be the application's formal name[…]. An example would be "FredApp E-Mail Client," rather than "fredapp-email-client."
So the dataengine only gets "KDE Power Management System" and not "powerdevil", so it has to match this manually. And also, I cannot inbeforehand tell the plasmoid if a config is available at all, ie. all notifications have the wrench but if the emitting application does not offer settings (Because it is a non-native KDE app or is not configurable, like KDialog) it just does nothing (or emits a messagebox, or so, but the button is still there).
For illustration how it works, see here: http://666kb.com/i/cdzg50m1huwc1k751.jpg ;-)</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;">Works, in both the popup and history. Tested with Amarok and Powerdevil.</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>plasma/generic/applets/notifications/contents/ui/LastNotificationPopup.qml <span style="color: grey">(df63ae6)</span></li>
<li>plasma/generic/applets/notifications/contents/ui/NotificationDelegate/NotificationDelegate.qml <span style="color: grey">(64d9298)</span></li>
<li>plasma/generic/applets/notifications/contents/ui/Notifications.qml <span style="color: grey">(c5be0a3)</span></li>
<li>plasma/generic/dataengines/notifications/CMakeLists.txt <span style="color: grey">(e8d66e0)</span></li>
<li>plasma/generic/dataengines/notifications/notificationaction.cpp <span style="color: grey">(390cc16)</span></li>
<li>plasma/generic/dataengines/notifications/notifications.operations <span style="color: grey">(bb5d0c6)</span></li>
<li>plasma/generic/dataengines/notifications/notificationsengine.h <span style="color: grey">(4644c8e)</span></li>
<li>plasma/generic/dataengines/notifications/notificationsengine.cpp <span style="color: grey">(77f5db2)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/110389/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>