Differential D27842

[breeze-icons] New 48px telegram icon
ClosedPublic
Actions

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

Details

Reviewers
ngraham
ndavis
Group Reviewers
VDG
Commits
R266:38dc1aa148d7: [breeze-icons] New 48px telegram icon
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.Mar 4 2020, 5:12 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptMar 4 2020, 5:12 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
rocka requested review of this revision.Mar 4 2020, 5:12 PM
ngraham accepted this revision.Mar 4 2020, 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.Mar 4 2020, 7:13 PM
ndavis accepted this revision.Mar 5 2020, 3:11 AM
Closed by commit R266:38dc1aa148d7: [breeze-icons] New 48px telegram icon (authored by ndavis). · Explain WhyMar 5 2020, 3:18 AM
This revision was automatically updated to reflect the committed changes.
ndavis added a comment.Mar 5 2020, 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.Mar 5 2020, 3:22 AM

Yes, I submitted it via the Web UI.

ndavis added a comment.Mar 5 2020, 3:25 AM
In D27842#622396, @rocka wrote:

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.

Revision Contents
Changeset List

PathPackages
AMicons-dark/apps/48/telegram.svg (17 lines)
AMicons/apps/48/telegram.svg (17 lines)

Diff 76986

View Options

icons-dark/apps/48/telegram.svg

Loading...
View Options

icons/apps/48/telegram.svg

Loading...