D28107: Fix overlayIcon sometimes not visible

[David Redondo]

davidre created this revision.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
davidre requested review of this revision.

REVISION SUMMARY
  Workaround a specific behavior in IconItem if the source is a QIcon with a name
  and pixmaps. IconItem will use the name to find the original svg in any case
  and render that. This causes the rendered pixmaps that include the overlay inside
  the icons to be ignored by IconItem. This happens only when the service has a
  IconName but no IconPixmap (because the data engine prefers that) and
  a OverlayIconName.

REPOSITORY
  R120 Plasma Workspace

BRANCH
  workaround (branched from master)

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

AFFECTED FILES
  applets/systemtray/package/contents/ui/items/StatusNotifierItem.qml
  applets/systemtray/systemtraymodel.cpp
  applets/systemtray/systemtraymodel.h

To: davidre
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200317/33229766/attachment.html>