Review Request: Implement simple Growl support in KNotify
Olivier Goffart
ogoffart at kde.org
Tue Aug 10 19:05:49 BST 2010
Le Tuesday 10 August 2010, Markus Slopianka a écrit :
> > On 2010-08-10 17:04:41, Olivier Goffart wrote:
> > > This is a bit hackish (the hardcoded paths, and the QSystemTrayIcon
> > > stuff) But if it works, i'm ok with it.
>
> Amateur question: Wouldn't is make more sense if a "wrapper service" was
> implemented that calls Growl instead of calling Growl directly from
> KNotify? That way hackish code wouldn't hurt.
KNotify is already a wrapper service. We do not need more :-)
More information about the kde-core-devel
mailing list