[Differential] [Commented On] D2092: Fix icon loading in Widget Explorer

[davidedmundson (David Edmundson)]

davidedmundson added a comment.


  I know, I copied the "is this icon valid" from there.
  
  I chose not to use it, partly because we don't have 5.7 and partly because it means you prematurely create the fallback qicon which is slightly additional overhead.
  
  I think related to this change the standard fromTheme behaviour must have changed. Though its a bit hard to see with all this gtk icon cache changes in the log.

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson
Cc: broulik, plasma-devel, jensreuterberg, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160705/8fd1516b/attachment.html>