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

Reply via email to