[Differential] [Request, 5 lines] D2092: Fix icon loading in Widget Explorer
davidedmundson (David Edmundson)
noreply at phabricator.kde.org
Tue Jul 5 17:08:02 UTC 2016
davidedmundson created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
In new Qt (not sure which version) QIcon::fromTheme(QString) returns a
valid icon object, thus hasThemeIcon always returns true as that simply
calls fromTheme internally and checks the names match.
This patch uses the same check as the new QIcon::fromTheme(QString
&name, QIcon fallback) to check the original icon was found.
This patch is therefore not only is a workaround but also a speed bost
as before we were effectively loading the same icon twice.
BUG: 365111
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
BRANCH
Plasma/5.7
REVISION DETAIL
https://phabricator.kde.org/D2092
AFFECTED FILES
components/shellprivate/widgetexplorer/plasmaappletitemmodel.cpp
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: davidedmundson
Cc: plasma-devel, jensreuterberg, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160705/3a030fa6/attachment.html>
More information about the Plasma-devel
mailing list