<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 />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 14th, 2013, 6:46 p.m. UTC, <b>Aaron J. Seigo</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">code looks fine, the feature i think will be welcome by many.
my only concern is how usable this will be on touch. (probably not very.) for Plasma Active 4 this is not a problem as it is using 4.10. We'll need to revisit this in later versions. (Probably with some use of the platformcomponents/ folder)</pre>
</blockquote>
<p>On May 14th, 2013, 7:01 p.m. UTC, <b>Thomas Pfeiffer</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Can't we use different designs for Desktop and Active?</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I thought too the notifications only used the stuff in the platformcontents directory which I didn't touch on purpose.
Given that Knotify doesnt have a touch config interface (yet), the thing will not be useful on touch at the moment, at least not provide a pleasant experience, I guess.</pre>
<br />
<p>- Kai Uwe</p>
<br />
<p>On May 12th, 2013, 2:10 p.m. UTC, Kai Uwe Broulik wrote:</p>
<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>
<p style="color: grey;"><i>Updated May 12, 2013, 2:10 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;">Depends on Review 110407
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 for permission to use his notifyrc lookup code that is licensed GPLv2 in this LGPLv2 code. We need to do this because 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 KNotify had to be modified to not only expose "KDE Power Management System" to the dataengine but also its internal name "powerdevil".
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: Shows configure button properly
- PowerDevil: Shows configure button properly
- Konsole: Shows configure button properly
- Free Space Notifier: Shows configure button properly
- KDialog: Does not show configure button, as expected
- QDBusViewer to issue a notification using the FDO DBus iface: Does not show configure button, as expected (it does, when your passed appName happens to match an application, which is rather nice, though)</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>
<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/05/12/notificationconfigure1.png">A few notifications with and without configure option</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>