Differential D21523

Add 22px notifications icons
ClosedPublic
Actions

Authored by ndavis on Jun 1 2019, 2:15 PM.

Details

Reviewers
filipf
ngraham
Group Reviewers
VDG
Commits
R266:cdd90177d393: Add 22px notifications icons
Summary

Added because 16px version of these icons are currently used in the notifications widget where 22px icons should be used

Test Plan

Diff Detail

Repository
R266 Breeze Icons
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ndavis created this revision.Jun 1 2019, 2:15 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptJun 1 2019, 2:15 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
ndavis requested review of this revision.Jun 1 2019, 2:15 PM
ndavis edited the test plan for this revision. (Show Details)Jun 1 2019, 2:16 PM
filipf accepted this revision.EditedJun 1 2019, 9:53 PM
filipf added a subscriber: filipf.

Looks good, but also just add a few words to the description about why we need this.

This revision is now accepted and ready to land.Jun 1 2019, 9:53 PM
ngraham accepted this revision.Jun 1 2019, 10:00 PM
ngraham added a subscriber: ngraham.

^^ Same

ndavis edited the summary of this revision. (Show Details)Jun 1 2019, 11:15 PM

done

Closed by commit R266:cdd90177d393: Add 22px notifications icons (authored by ndavis). · Explain WhyJun 1 2019, 11:17 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 58997

View Options

icons-dark/actions/22/notifications-disabled.svg

Loading...
View Options

icons-dark/actions/22/notifications.svg

Loading...
View Options

icons/actions/22/notifications-disabled.svg

Loading...
View Options

icons/actions/22/notifications.svg

Loading...