D14856: Search: Add workaround for missing icons in Gnome icon-theme

Kai Uwe Broulik noreply at phabricator.kde.org
Wed Aug 15 09:54:50 BST 2018


broulik 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::hasThemeIcon(QStringLiteral("games-config-options")) ? QIcon::fromTheme(QStringLiteral("games-config-options")) : QIcon::fromTheme(QStringLiteral("preferences-system"));
> +        QIcon matchCaseIcon = QIcon::hasThemeIcon(QStringLiteral("format-text-superscript")) ? QIcon::fromTheme(QStringLiteral("format-text-superscript")) : QIcon::fromTheme(QStringLiteral("format-text-bold"));

`hasThemeIcon)foo)` just does `!QIcon::fromTheme(foo).isNull()` so you might as well just do the same and safe a lookup

REPOSITORY
  R39 KTextEditor

BRANCH
  master

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

To: sars, #kate, dhaumann
Cc: broulik, dhaumann, kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns, demsking, cullmann, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180815/d70d8cd5/attachment.html>


More information about the Kde-frameworks-devel mailing list