davidre added a comment.
In D27617#630440 <https://phabricator.kde.org/D27617#630440>, @ilya-fedin wrote: > In D27617#629987 <https://phabricator.kde.org/D27617#629987>, @ngraham wrote: > > > Would it help this use case if we moved the Telegram icon into the Plasma theme for now, pending a fix in Telegram itself? > > > There is nothing to do on Telegram side. Everything that Telegram can get is icon theme name (sometime), does the icon exist, the icon itself and its sizes. Adding a dependency on kde frameworks is not a way, of course. > > In D27617#630392 <https://phabricator.kde.org/D27617#630392>, @IlyaBizyaev wrote: > > > Btw that overlay is always as tiny as on your screenshot, so not usable for a counter > > > +1. Moreover, icon is not recolored even with IconName + OverlayIconPixmap. Oh yes that's true :/. It's a similiar bug to the one I fixed not long ago. It seems the SNI desperately needs the change as outlined in the other diff by removing all the rendering from the data engine. This way also the sizing of the overlay, I think I noticed it being inconsistent between pixmap and overlayname path. 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
