[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    =?utf-8?q?=5Bkde-workspace=5D_plasma/generic/applets/notificatio?=
From:       Artur Duque de Souza <asouza () kde ! org>
Date:       2011-04-28 19:23:20
Message-ID: 20110428192320.CF1E4A60A4 () git ! kde ! org
[Download RAW message or body]

Git commit 71642957d337be6e9327b10e618b4d7927eed38e by Artur Duque de Souza.
Committed on 28/04/2011 at 21:18.
Pushed by asouza into branch 'master'.

Automatically delete the notification when action clicked

When the user clicks on the button that will trigger the action of
the notification, we delete the notification so it doesn't hang there
'forever'. Let's see how it goes, but I think it improves the usability.

CCMAIL:plasma-devel@kde.org

Signed-off-by: Artur Duque de Souza <asouza@kde.org>
Reviewed-by: Marco Martin <mart@kde.org>

M  +1    -1    plasma/generic/applets/notifications/ui/notificationwidget.cpp     

http://commits.kde.org/kde-workspace/71642957d337be6e9327b10e618b4d7927eed38e

diff --git a/plasma/generic/applets/notifications/ui/notificationwidget.cpp \
b/plasma/generic/applets/notifications/ui/notificationwidget.cpp index \
                3f87cfe..72bb108 100644
--- a/plasma/generic/applets/notifications/ui/notificationwidget.cpp
+++ b/plasma/generic/applets/notifications/ui/notificationwidget.cpp
@@ -433,7 +433,7 @@ void NotificationWidgetPrivate::buttonClicked()
 {
     //a decsion has already been taken
     if (actionsWidget) {
-        actionsWidget->hide();
+        notification.data()->deleteLater();
     }
     emit q->actionTriggered(notification.data());
 }
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel


[prev in list] [next in list] [prev in thread] [next in thread]