[Notification Popup] Clear notification data after popup closed
ClosedPublic

Authored by broulik on Dec 1 2016, 2:41 PM.

Details

Summary

There's no point in keeping around the notification data until it is replaced by a new one.

Test Plan

Verified that e.g. the thumbnail strip is destroyed when the popup closes. Also keeps it from doing stupid things (like requesting new thumbnails while closing) etc.

Might also reduce teh likeliehood of layout glitches when we always start with a "fresh" notification content rather than some old actions and what not hanging around. Layouts are quite fragile...

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik updated this revision to Diff 8676.Dec 1 2016, 2:41 PM
broulik retitled this revision from to [Notification Popup] Clear notification data after popup closed.
broulik updated this object.
broulik edited the test plan for this revision. (Show Details)
broulik added a reviewer: Plasma.
broulik set the repository for this revision to R120 Plasma Workspace.
Restricted Application added a project: Plasma. · View Herald TranscriptDec 1 2016, 2:41 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart accepted this revision.Dec 5 2016, 10:25 AM
mart added a reviewer: mart.
This revision is now accepted and ready to land.Dec 5 2016, 10:25 AM
This revision was automatically updated to reflect the committed changes.