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

[Ilya Bizyaev]

IlyaBizyaev added subscribers: ilya-fedin, davidre.
IlyaBizyaev added a comment.


  In D27617#630228 <https://phabricator.kde.org/D27617#630228>, @ngraham wrote:
  
  > So... how do we fix this so that you can use a nice monochrome Breeze icon in your system tray? Whose code needs to change?
  
  
  
  
  1. SNI implementation in Plasma needs to be fixed to properly support OverlayIconPixmap <https://www.freedesktop.org/wiki/Specifications/StatusNotifierItem/StatusNotifierItem/#org.freedesktop.statusnotifieritem.overlayiconpixmap> with IconName. From what I understand, since it affects a data engine and that is considered public API, this might have to wait for Plasma 6
  2. Since old versions of Plasma will still have the issue, and other SNI implementations likely don't support this either, this can't become the default behavior for Telegram; so there has to be a way for Telegram to check if an SNI implementation supports what it needs
  3. Then Telegram code needs to add a special case for that.
  
  Since I feel like a proxy at this point, tagging @davidre and @ilya-fedin.

REPOSITORY
  R266 Breeze Icons

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

To: rocka, #vdg, Fuchs, ndavis
Cc: davidre, ilya-fedin, broulik, alexeymin, IlyaBizyaev, 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/20200319/935bffe6/attachment.html>