D27787: Remove 48px telegram icons

[Nathaniel Graham]

ngraham added a comment.


  Here's one problem from simply removing the existing Breeze theme SVG icons: 
the icons shipped in Telegram are raster .pngs which can scale poorly, and the 
circular effect seems quite badly done for the 48px version in particular. For 
example here's how Telegram's icon now looks in my 60px thickness panel with an 
IOTM:
  
  F8149572: Screenshot_20200304_090413.png 

  
  This is Telegram's fault, of course. But it's still gonna bite our users 
until they fix it.

REPOSITORY
  R266 Breeze Icons

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

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

D27787: Remove 48px telegram icons

[Noah Davis]

ndavis added a comment.


  In D27787#621050 , @rocka wrote:
  
  > Could we please not remove this icon, or at least keep it available as an 
alternative? It fits the general style so well.
  
  
  The only way to have alternative icons is to make an alternative icon theme 
that uses the alternative icon as its only icon.
  
  This is close enough to being a breeze icon that we really don't need our own 
version of it for now: 
https://raw.githubusercontent.com/telegramdesktop/tdesktop/dev/Telegram/Resources/art/icon512.png
  
  If the desktop client icon changes to an incompatible style, we'll make a new 
breeze telegram icon.

REPOSITORY
  R266 Breeze Icons

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

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

D27787: Remove 48px telegram icons

[Rocket Aaron]

rocka added a comment.


  Could we please not remove this icon, or at least keep it available as an 
alternative? It fits the general style so well.

REPOSITORY
  R266 Breeze Icons

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

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

D27787: Remove 48px telegram icons

[Noah Davis]

This revision was not accepted when it landed; it landed in state "Needs 
Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit R266:8fb85eb7cca0: Remove 48px telegram icons (authored by 
ndavis).

REPOSITORY
  R266 Breeze Icons

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D27787?vs=76782=76794

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

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

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

D27787: Remove 48px telegram icons

[Noah Davis]

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

REVISION SUMMARY
  The style doesn't preserve the original branding very well and the official 
telegram icon is close enough to being a Breeze icon that we don't need a 
replacement icon.

REPOSITORY
  R266 Breeze Icons

BRANCH
  remove-telegram (branched from master)

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

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

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