From 8315831245ede84fa716c0e4192e83a25d2563d4 Mon Sep 17 00:00:00 2001 From: Kai Uwe Broulik Date: Sat, 8 Nov 2014 13:45:56 +0100 Subject: [PATCH] Fix notification configure button Unfortunately we cannot make the systray popup does not close when the config dialog appears --- .../package/contents/ui/NotificationDelegate.qml | 1 - .../notifications/package/contents/ui/NotificationPopup.qml | 5 +++++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/applets/notifications/package/contents/ui/NotificationDelegate.qml b/applets/notifications/package/contents/ui/NotificationDelegate.qml index 07fa2d408..e05cda858 100644 --- a/applets/notifications/package/contents/ui/NotificationDelegate.qml +++ b/applets/notifications/package/contents/ui/NotificationDelegate.qml @@ -168,7 +168,6 @@ PlasmaComponents.ListItem { } } onConfigure: { - plasmoid.hidePopup() configureNotification(model.appRealName) } onAction: { diff --git a/applets/notifications/package/contents/ui/NotificationPopup.qml b/applets/notifications/package/contents/ui/NotificationPopup.qml index 2611e6911..743af4446 100644 --- a/applets/notifications/package/contents/ui/NotificationPopup.qml +++ b/applets/notifications/package/contents/ui/NotificationPopup.qml @@ -92,6 +92,7 @@ PlasmaCore.Dialog { text: notificationProperties ? notificationProperties.body : "" icon: notificationProperties ? notificationProperties.appIcon : "" image: notificationProperties ? notificationProperties.image : undefined + configurable: notificationProperties ? notificationProperties.configurable : false anchors { fill: parent @@ -110,6 +111,10 @@ PlasmaCore.Dialog { closeNotification(notificationProperties.source) notificationPopup.hide() } + onConfigure: { + configureNotification(notificationProperties.appRealName) + notificationPopup.hide() + } onAction: { executeAction(notificationProperties.source, actionId) actions.clear()