[breeze-icons] New 48px telegram icon
ClosedPublic

Authored by rocka on Wed, Mar 4, 5:12 PM.

Details

Summary

Since 48px telegram icon was removed in D27787, I made a new one following the official icon.

Test Plan

256x256 PNG:

In panel with other icons:

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.
rocka created this revision.Wed, Mar 4, 5:12 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptWed, Mar 4, 5:12 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
rocka requested review of this revision.Wed, Mar 4, 5:12 PM
ngraham accepted this revision.Wed, Mar 4, 7:13 PM
ngraham added a reviewer: ndavis.
ngraham added a subscriber: ngraham.

This looks great to me! IMO it's definitely better than the original, but in subtle ways.

This revision is now accepted and ready to land.Wed, Mar 4, 7:13 PM
ndavis accepted this revision.Thu, Mar 5, 3:11 AM
This revision was automatically updated to reflect the committed changes.
ndavis added a comment.Thu, Mar 5, 3:21 AM

That's weird. It should say authored by @rocka, not me. @rocka, how did you submit this patch? Did you submit it via the web UI?

rocka added a comment.Thu, Mar 5, 3:22 AM

Yes, I submitted it via the Web UI.

ndavis added a comment.Thu, Mar 5, 3:25 AM

Yes, I submitted it via the Web UI.

:/ that happens when people submit via the web UI for whatever reason. It'll no longer be an issue when we move KF5 to GitLab, but try to use arcanist (arc) in the future.