Use new icon for clearing notifications
ClosedPublic

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

Diff Detail

Repository
R224 KDE Connect
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
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 !