D5109: [Kicker/App Entry] Try QIcon with path if no theme icon is found

David Edmundson noreply at phabricator.kde.org
Tue Oct 31 22:56:44 UTC 2017


davidedmundson added a comment.


  > KIconLoader does, but even Qt's own ones QSvgIconEngine and QPixmapIconEngine don't currently.
  
  I did fix QPixmapIcon and QSvgIconEngine,
  
  This should mean we can change QIcon::fromTheme to check isNull() which will then implicitly solve this.

REPOSITORY
  R119 Plasma Desktop

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

To: broulik, #plasma, hein
Cc: rkflx, smartins, ngraham, davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171031/2de70afd/attachment.html>


More information about the Plasma-devel mailing list