-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5364/#review9932
-----------------------------------------------------------


I would love to see this integrated. Any reasons for not shipping it?

- Cyberbeat


On Sept. 16, 2010, 2:20 p.m., Igor Poboiko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5364/
> -----------------------------------------------------------
> 
> (Updated Sept. 16, 2010, 2:20 p.m.)
> 
> 
> Review request for Kopete.
> 
> 
> Summary
> -------
> 
> This patch makes user_is_typing notifications to use ActiveNotifications. I 
> did it because kopete sometimes shows up to five different user_is_typing 
> notifications for one user (as for me) simultaneously, and it really annoys 
> me sometimes. With it, kopete will show only one this notification for an 
> user in one time.
> It also adds an user_is_typing_message notification for oscar protocol even 
> if there are no available chat session for current contact (for example, 
> someone is typing a message to you first time)
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdenetwork/kopete/libkopete/kopetechatsession.h 1175782 
>   /trunk/KDE/kdenetwork/kopete/libkopete/kopetechatsession.cpp 1175782 
>   /trunk/KDE/kdenetwork/kopete/libkopete/private/kopeteactivenotification.h 
> 1175782 
>   /trunk/KDE/kdenetwork/kopete/libkopete/private/kopeteactivenotification.cpp 
> 1175782 
>   /trunk/KDE/kdenetwork/kopete/protocols/oscar/oscarcontact.cpp 1175782 
> 
> Diff: http://svn.reviewboard.kde.org/r/5364/diff
> 
> 
> Testing
> -------
> 
> Patch applies to current trunk. 
> Kopete compiles fine with it. 
> If someone (oscar contact) starts typing you a message (if there are no 
> available windows with this contact), it shows user_is_typing notification.
> If there is one visible user_is_typing notification, kopete doesn't show a 
> new notification.
> 
> So all works fine now (as for me, of course)
> 
> 
> Thanks,
> 
> Igor
> 
>

_______________________________________________
kopete-devel mailing list
kopete-devel@kde.org
https://mail.kde.org/mailman/listinfo/kopete-devel

Reply via email to