D27617: [breeze-icons] Add telegram-desktop tray icons

[Noah Davis]

ndavis added a comment.


  Sorry, I've been busy lately.
  
  I noticed some issues:
  
  1. The distinguishing part of the icon doesn't take up much space, so too much of it gets hidden behind the notification badge.
    - I'd suggest removing the circle and making the distinguishing part larger.
  2. The style should match the official logo better. I'm removing the 48px breeze icons in D27787 <https://phabricator.kde.org/D27787> since they're outdated and the official icon is extremely close to being a Breeze icon.
    - You could just copy the relevant bits from Papirus icons and resize/edit them for Breeze icons.
  3. Since I'm removing the 48px Breeze icons, you'll have to make a patch to the desktoptheme in plasma-frameworks instead of Breeze icons or else breeze will use monochrome icons for telegram everywhere. Sorry for the inconvenience, but feel free to message me on Matrix (`@noahdvs:kde.org`) or Telegram (`@noahdvs`) if you need help.

REPOSITORY
  R266 Breeze Icons

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

To: rocka, #vdg, Fuchs
Cc: ndavis, ngraham, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200302/d8fc57f6/attachment.html>