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