From kopete-devel Mon Mar 17 22:08:00 2008 From: "Roman Jarosz" Date: Mon, 17 Mar 2008 22:08:00 +0000 To: kopete-devel Subject: Re: [kopete-devel] [PATCH] Non-intrusive notifications v3 Message-Id: X-MARC-Message: https://marc.info/?l=kopete-devel&m=120579177801193 On Mon, 17 Mar 2008 21:59:43 +0100, Olivier Goffart wrote: > Hi, > > Thanks for the patch. > > > Le dimanche 16 mars 2008, Roman Jarosz a écrit : >> Hi >> >> Here is the final version of non-intrusive notifications. >> >> Since last version I have added KNotification for new info events >> (the KNotification name is "Service Message") > > We probably want different event for eatch type of event :-) Personally I don't need more than one notification for Service Messages. Let see what other thinks? > >> and made some slight >> adjustment to the API. I've also fixed couple bug and polished the gui. >> >> All protocols are now converted to the new API, I didn't test the Yahoo, >> MSN and Telepaty but it should be ok. If anybody can test these protocols >> please do. >> >> If nobody objects I'll probably commit it tomorrow. >> >> Thanks, >> Roman > > > I don't see why you put events in a separate directory, but that doesn't > matter to me. Should I put it into the libkopete/ui or into libkopete? > Jabber: > >> void JabberAccount::slotAddedInfoEventActionActivated ( uint actionId ) >> { [...] >> if ( actionId == Kopete::AddedInfoEvent::AuthorizeAction ) >> { [...] } >> else if ( actionId == Kopete::AddedInfoEvent::BlockAction ) >> { [...] } >> else if( actionId == Kopete::AddedInfoEvent::AddContactAction ) >> { [...] } > > There is two issues. > 1) You may want to authorize the contact without adding it (specially if the > contact was already on the list, but 'blocked'). Or even adding it without > authorizing it. ("authorize" here mean allowing the user to see the > presence) > 2) if you go that way you should at least authorize when adding. > > I think this is a design problem on the new dialog. > (And I think the problem is the same on MSN) No this isn't problem: If you click authorize then AuthorizeAction is emitted. If you choose "Add..." the dialog is opened. If authorize is check in add dialog then AuthorizeAction is emitted. If add is check AddContactAction is emitted. If both are check then first AuthorizeAction and than AddContactAction are emitted. I'll add note that actionActivated can be emitted more than once. >> +#include > uh? :-) ohh!! how this got in :) Thanks for noticing. _______________________________________________ kopete-devel mailing list kopete-devel@kde.org https://mail.kde.org/mailman/listinfo/kopete-devel