D8060: allow to set a custom palette instead of colorSets

Kai Uwe Broulik noreply at phabricator.kde.org
Sun Oct 1 09:14:27 UTC 2017


broulik added inline comments.

INLINE COMMENTS

> kiconloader.cpp:869
>             % QLatin1Char('_')
> -           % paletteId(qApp->palette(), mColorSet)
> +           % paletteId(mPalette)
>             % (q->theme() && q->theme()->followsColorScheme() && state == KIconLoader::SelectedState ? QStringLiteral("_selected") : QString());

You don't seem to be following `qApp->palette()` anymore if no custom one is set?

REPOSITORY
  R302 KIconThemes

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

To: mart, dfaure, #frameworks, rikmills
Cc: broulik, cfeck, #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171001/804c0297/attachment.html>


More information about the Kde-frameworks-devel mailing list