D27842: [breeze-icons] New 48px telegram icon

[Noah Davis]

ndavis added a comment.


  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.

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D27842

To: rocka, #vdg, ngraham, ndavis
Cc: ngraham, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, bruns

D27842: [breeze-icons] New 48px telegram icon

[Rocket Aaron]

rocka added a comment.


  Yes, I submitted it via the Web UI.

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D27842

To: rocka, #vdg, ngraham, ndavis
Cc: ngraham, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, bruns

D27842: [breeze-icons] New 48px telegram icon

[Noah Davis]

ndavis added a comment.


  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?

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D27842

To: rocka, #vdg, ngraham, ndavis
Cc: ngraham, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, bruns

D27842: [breeze-icons] New 48px telegram icon

[Noah Davis]

This revision was automatically updated to reflect the committed changes.
Closed by commit R266:38dc1aa148d7: [breeze-icons] New 48px telegram icon 
(authored by ndavis).

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27842?vs=76956=76986

REVISION DETAIL
  https://phabricator.kde.org/D27842

AFFECTED FILES
  icons-dark/apps/48/telegram.svg
  icons/apps/48/telegram.svg

To: rocka, #vdg, ngraham, ndavis
Cc: ngraham, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, bruns

D27842: [breeze-icons] New 48px telegram icon

[Noah Davis]

ndavis accepted this revision.

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D27842

To: rocka, #vdg, ngraham, ndavis
Cc: ngraham, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, bruns

D27842: [breeze-icons] New 48px telegram icon

[Nathaniel Graham]

ngraham accepted this revision.
ngraham added a reviewer: ndavis.
ngraham added a comment.
This revision is now accepted and ready to land.


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

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D27842

To: rocka, #vdg, ngraham, ndavis
Cc: ngraham, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, bruns

D27842: [breeze-icons] New 48px telegram icon

[Rocket Aaron]

rocka created this revision.
rocka added a reviewer: VDG.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
rocka requested review of this revision.

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

TEST PLAN
  256x256 PNG:
  
  F8149657: out.png 
  
  In panel with other icons:
  
  F8149659: image.png 

REPOSITORY
  R266 Breeze Icons

REVISION DETAIL
  https://phabricator.kde.org/D27842

AFFECTED FILES
  icons-dark/apps/48/telegram.svg
  icons/apps/48/telegram.svg

To: rocka, #vdg
Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns