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

[David Redondo]

davidre added a comment.


  In D27617#630336 <https://phabricator.kde.org/D27617#630336>, @IlyaBizyaev wrote:
  
  > 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.
  
  
  Huh that case is working what I found that `IconName` and `OverlayIconName` is currently broken, see D28107 <https://phabricator.kde.org/D28107>.
  `IconName and `OverlayIconPixmap` work fine  here F8184158: Screenshot_20200319_102827.PNG <https://phabricator.kde.org/F8184158>
  
  If it doesn't for you please file a bug so we can investigate that.

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/f42de096/attachment-0001.html>