D14856: Search: Add workaround for missing icons in Gnome icon-theme
Stefan BrĂ¼ns
noreply at phabricator.kde.org
Wed Aug 15 17:07:15 BST 2018
bruns added inline comments.
INLINE COMMENTS
> katesearchbar.cpp:1332
> + // Gnome does not seem to have all icons we want, so we use fall-back icons for those that are missing.
> + QIcon mutateIcon = QIcon::fromTheme(QStringLiteral("games-config-options"), QIcon::fromTheme(QStringLiteral("preferences-system")));
> + QIcon matchCaseIcon = QIcon::fromTheme(QStringLiteral("format-text-superscript"), QIcon::fromTheme(QStringLiteral("format-text-bold")));
Doesn't this evaluate the fallback always, even if not needed?
IIRC QIcon::fromTheme is not really lightweight ...
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D14856
To: sars, #kate, dhaumann, broulik
Cc: bruns, broulik, dhaumann, kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, demsking, cullmann, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180815/2765c257/attachment.html>
More information about the Kde-frameworks-devel
mailing list