davidre added inline comments.

INLINE COMMENTS

> breezebutton.cpp:148
> +            QPalette palette = decoration()->client().data()->palette();
> +            palette.setColor(QPalette::Foreground, 
> qobject_cast<Decoration*>(decoration())->fontColor());
> +            KIconLoader::global()->setCustomPalette(palette);

I don't think it can fail. decoration() is a QPointer 
(https://api.kde.org/kdecoration/html/classKDecoration2_1_1DecorationButton.html)
 do you mean that with cross-cast?

REPOSITORY
  R31 Breeze

BRANCH
  icons (branched from master)

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

To: davidre, #plasma, #vdg, ngraham
Cc: kossebau, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to