[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    An action in KNotify
From:       Olivier Goffart <ogoffart () tiscalinet ! be>
Date:       2003-08-19 7:53:56
[Download RAW message or body]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1


Hello,

I coded, for Kopete, a nice feature in KNotify.
It allow us to add an action in KNotify in visual effect (a link in the 
passive popup, or a button in the messagebox)

it looks like this:
http://gof.homeip.net/olivier/passivepopup.png

The action is handled by the application.


To do that, i modified the knotifyclient. I commited it in 
kdenonbeta/kopete/libkopete/compat
http://webcvs.kde.org/cgi-bin/cvsweb.cgi/~checkout~/kdenonbeta/kopete/libkopete/compat/knotifyclient.h?rev=1.1&content-type=text/plain
 http://webcvs.kde.org/cgi-bin/cvsweb.cgi/~checkout~/kdenonbeta/kopete/libkopete/compat/knotifyclient.cpp?rev=1.1&content-type=text/plain


the notify call looks like this:
  KNotifyClient:: event( winId, message, text , kguiitem, qobject, SLOT());


Technicaly, i moved some code from the knotify deamon to the knotifyclient 
itself. I left just the sound notification to the deamon. It is simply 
because i did not found simple way to translate a qt slot via DCop safely.


I would like to add it in the kdelibs.
However, we can't place it in kdecore because it use some kdewidget (like 
kmessagebox, kguiitem, kpassivepopup and kactivelabel)
And moving it to kdeui is maybe not a solution. kcompletion in kdecore uses 
the knotifyclient.


Anyway i think it is a great addition, and a don't want to have such as 
duplicate code in kopete.

What do you think?


(sorry for my bad english)

- --
Olivier 
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)

iD8DBQE/QdeYz58lY8jWrL0RAp0JAJ90YmTYwu7hof0K+Iy7onKGzp3XLQCeKjRt
HudpcayozXUOxSFRBP6qjNQ=
=fP8N
-----END PGP SIGNATURE-----

 
> > Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


[prev in list] [next in list] [prev in thread] [next in thread]