[Notifications] Let clear entry also clear history
ClosedPublic

Authored by broulik on Feb 5 2018, 1:33 PM.

Details

Summary

There's a "Clear Notifications" context menu entry. So far it would only show up and clear persistent notifications. The main usecase for this should be to clear the history, however. This patch makes it do both.

Test Plan

Enabled history, spawned a notification, "Clear notifications" context menu entry showed up and cleared it.
Spawned a persistent notification, same.
Both also worked.

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 created this revision.Feb 5 2018, 1:33 PM
Restricted Application added a project: Plasma. · View Herald TranscriptFeb 5 2018, 1:33 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Feb 5 2018, 1:33 PM
mart accepted this revision.Feb 6 2018, 10:15 AM
This revision is now accepted and ready to land.Feb 6 2018, 10:15 AM
This revision was automatically updated to reflect the committed changes.