broulik created this revision. broulik added reviewers: Plasma, VDG. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. broulik requested review of this revision.
REVISION SUMMARY Also consider a notification as unread only when it isn't active anymore. TEST PLAN `notify-send foo bar`, waited for popup to close, got a "1" in my panel `notify-send foo bar`, closed the popup, didn't get anything in my panel As suggested by VDG I originally had the notification plasmoid show up alongside the popup to kinda indicate "this is where this thing is coming from/going to". With this change the notification plasmoid will not be shown in the panel unless there is an unread notification. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D21423 AFFECTED FILES applets/notifications/package/contents/ui/CompactRepresentation.qml libnotificationmanager/notifications.cpp To: broulik, #plasma, #vdg Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
