Bug 321688

Summary: KNotification::event() should support QIcon/QImage in addition to QPixmap
Product: [Frameworks and Libraries] frameworks-knotifications Reporter: Christoph Feck <cfeck>
Component: generalAssignee: Martin Klapetek <mklapetek>
Status: RESOLVED FIXED    
Severity: wishlist CC: simonandric5
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:

Description Christoph Feck 2013-06-27 22:38:10 UTC
There was a recent commit changing a Nepomuk server tool to use QApplication instead of QCoreApplication, just because it wanted to use KNotification::event() to send events.

Such events are sent over to the knotify daemon via D-Bus, and if the application wants to display an icon, it currently has to use QPixmap.

KNotification::event() should additionally support QIcon and/or QImage for its icons, so that such a server tool can stay a QCoreApplication.

See bug 321499 for an example.
Comment 1 Marco Martin 2014-10-29 13:56:51 UTC
Git commit fc9bfbc78641008d0ca7b303bb0547c73b3e29bd by Marco Martin.
Committed on 29/10/2014 at 13:55.
Pushed by mart into branch 'master'.

add a new KNotification::Event with icon name

add a convenience call to create knotifications with an icon name
REVIEW:120888

M  +15   -0    src/knotification.cpp
M  +22   -1    src/knotification.h

http://commits.kde.org/knotifications/fc9bfbc78641008d0ca7b303bb0547c73b3e29bd