apol added inline comments. INLINE COMMENTS
> jucato wrote in knotification.h:96 > Maybe we should keep this part that the icon name, especially that it has to > be one that can be found by KIconLoader. Since Qt5 KIconLoader is an implementation detail. Even our apps and frameworks use QIcon::fromTheme, I haven't looked at what KNotifications does but pretty sure it's using QIcon. REPOSITORY R289 KNotifications REVISION DETAIL https://phabricator.kde.org/D26918 To: nicolasfella, #frameworks, broulik, jucato Cc: class, apol, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns