diff --git a/applets/notifications/package/contents/ui/NotificationPopup.qml b/applets/notifications/package/contents/ui/NotificationPopup.qml index 15a0d2fa7..979d34b75 100644 --- a/applets/notifications/package/contents/ui/NotificationPopup.qml +++ b/applets/notifications/package/contents/ui/NotificationPopup.qml @@ -90,7 +90,7 @@ PlasmaCore.Dialog { } closeNotification(notificationProperties.source) - notificationPopup.hide() + // the popup will be closed in response to sourceRemoved } onContainsMouseChanged: { if (containsMouse) { @@ -126,22 +126,19 @@ PlasmaCore.Dialog { onClose: { closeNotification(notificationProperties.source) - notificationPopup.hide() + // the popup will be closed in response to sourceRemoved } onConfigure: { configureNotification(notificationProperties.appRealName, notificationProperties.eventId) - notificationPopup.hide() + notificationPositioner.closePopup(notificationProperties.source); } onAction: { executeAction(notificationProperties.source, actionId) actions.clear() - notificationPopup.hide() } onOpenUrl: { Qt.openUrlExternally(url) - // we want to close the notification popup when opening a URL but - // it should not be removed from the history, hence pretending it expired - notificationTimer.triggered() + notificationPositioner.closePopup(notificationProperties.source); } } } diff --git a/applets/notifications/package/contents/ui/Notifications.qml b/applets/notifications/package/contents/ui/Notifications.qml index 6826938cc..d0f253408 100644 --- a/applets/notifications/package/contents/ui/Notifications.qml +++ b/applets/notifications/package/contents/ui/Notifications.qml @@ -89,8 +89,9 @@ Column { //try to open the id as url } else if (id.indexOf("jobUrl#") !== -1) { Qt.openUrlExternally(id.slice(7)); - notificationPositioner.closePopup(source); } + + notificationPositioner.closePopup(source); } function configureNotification(appRealName, eventId) {