broulik added inline comments.

INLINE COMMENTS

> apol wrote in knotification.h:96
> 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.

It actually sends out the icon name as a string and plasmashell then looks it 
up.

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

Reply via email to