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

Reply via email to