[Notifications] Never manually hide() the NotificationPopup

Authored by broulik on Feb 17 2017, 9:34 AM.

Description

[Notifications] Never manually hide() the NotificationPopup

Otherwise the positioner will get completely confused.
When a notification is closed, the popup will already be closed in response to sourceRemoved.
Also, when triggering an action, call closePopup instead of hiding the popup.

(cherry picked from commit b97fdfa293dd4de59046c19f08aa6a3b940a790c)

Differential Revision: https://phabricator.kde.org/D4632

Details

Committed
broulikFeb 17 2017, 9:34 AM
Differential Revision
D4632: [Notifications] Never manually hide() the NotificationPopup
Parents
R120:62b803b45dbe: Add comment laying out goals.
Branches
Unknown
Tags
Unknown