D12459: [Icon KCM] Port to new design

[Friedrich W. H. Kossebau]

kossebau added inline comments.

INLINE COMMENTS

> main.cpp:393
> +        for (int i = 0; i < KIconLoader::LastGroup; i++) {
> +            KIconLoader::emitChange(KIconLoader::Group(i));
> +

Is KIconLoader::emitChange also for kde4 apps only? Asking as KIconLoader of KIconThemes seems to have this as part of normal API, without any deprecation notes, and e.g. the respective client-side signal is used in KXmlGui and other places.

What would be the Qt5/KF5 way otherwise?

REPOSITORY
  R119 Plasma Desktop

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

To: broulik, #plasma, #vdg, hein
Cc: kossebau, lueck, schwarzer, hein, zzag, abetts, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200425/e9475b4c/attachment-0001.html>