Use new icon for clearing notifications

Authored by nicolasfella on Oct 4 2018, 5:22 PM.

Diff Detail

R224 KDE Connect
Lint OK
No Unit Test Coverage
Build Status
Buildable 3497
Build 3515: arc lint + arc unit
nicolasfella created this revision.Oct 4 2018, 5:22 PM
Restricted Application added a project: KDE Connect. · View Herald TranscriptOct 4 2018, 5:22 PM
Restricted Application added a subscriber: kdeconnect. · View Herald Transcript
nicolasfella requested review of this revision.Oct 4 2018, 5:22 PM
nicolasfella edited the summary of this revision. (Show Details)Oct 4 2018, 5:23 PM
ngraham accepted this revision.Oct 4 2018, 5:24 PM

+1, this makes it consistent with the regular old notification widget.

This revision is now accepted and ready to land.Oct 4 2018, 5:24 PM
This revision was automatically updated to reflect the committed changes.

I am not a huge fan of the broom icon, but it's probably due to a cleaning disorder.
+1 for the consistency with plasma notification history clearing !