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

Ship it!


Assuming Kopete doesn't keep binary compatibility between releases, this can go 
in (remember to backport to the 4.5 and 4.6 branches). If you prefer to go on 
the safe side, do as suggested and keep BC.

- Raphael


On 2010-03-12 15:03:03, Teemu Rytilahti wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/3260/
> -----------------------------------------------------------
> 
> (Updated 2010-03-12 15:03:03)
> 
> 
> Review request for Kopete.
> 
> 
> Summary
> -------
> 
> This patch modifies the behaviour of idle timer to not to confirm whether to 
> come back from auto-away when there are no accounts online. The problem 
> before was that I got spammed "Do you want to set yourself as available" when 
> coming back from away, even if the Kopete was not connected to any servers.
> 
> As I am not familiar with the codebase, I hope someone can review and see 
> whether the approach I'm using here is ok.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdenetwork/kopete/libkopete/kopeteaccountmanager.h 1101310 
>   /trunk/KDE/kdenetwork/kopete/libkopete/kopetestatusmanager.h 1101310 
>   /trunk/KDE/kdenetwork/kopete/libkopete/kopetestatusmanager.cpp 1101310 
> 
> Diff: http://svn.reviewboard.kde.org/r/3260/diff
> 
> 
> Testing
> -------
> 
> Tested by using the patched version for a while. Returning from auto-away 
> still works just fine when there are accounts connected, but you won't get 
> confirmation dialog when coming back when there is no online accounts.
> 
> 
> Thanks,
> 
> Teemu
> 
>

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

Reply via email to