From kopete-devel  Tue Apr 22 21:13:26 2008
From: Roman Jarosz <R.Jarosz () sh ! cvut ! cz>
Date: Tue, 22 Apr 2008 21:13:26 +0000
To: kopete-devel
Subject: Re: [kopete-devel] [PATCH] Lid auto away
Message-Id: <200804222313.27171.R.Jarosz () sh ! cvut ! cz>
X-MARC-Message: https://marc.info/?l=kopete-devel&m=120889885320332

On Tuesday 22 of April 2008 22:01:28 Olivier Goffart wrote:
> Le mardi 22 avril 2008, Martijn Klingens a écrit :
> > On Saturday 19 April 2008 20:29:03 Roman Jarosz wrote:
> > > Any comments before I commit it?
> >
> > This forces the status on return to Online. Instead, you should restore
> > the previous status, whatever it was (especially if it was a manual Away,
> > or Do Not Disturb and/or it had a status message set).
> >
> > Also, do you want to override DnD or Invisible with Away?
>
> I don't think the patch change the current behaviour, which was correct
> (ie only set away accounts that were Online)

Yes, the behaviour wasn't changed.

> I just wonder what the d->autoAcounts.clear() are for.
d->autoAcounts contains accounts which were changed, so we can change them 
back. Clear is there because we have to clear d->autoAcounts when user sets 
global status while he's idle (e.g with dbus).

> > Last, I agree with Michal that if you simply lock the screen on lid close
> > you get this feature for free. You probably want that anyway, since a
> > blank screen avoids wasting cpu cycles on screen updates and thus is
> > better for your battery. With that in mind I think this feature belongs
> > in an external DBUS script or a plugin, but not libkopete.
Well the status won't be changed immediately after user locks the screen but 
after the auto away timeout.

> At least, I think it should be moved in Kopete::IdleTimer
> (But I would say the feature is not that great to have in libkopte either,
> althought i'm not strongly opposed to it)

I'm not against moving it to IdleTimer.
I'll wait and see what others think. ( if it should go in kdelibs or plugin )

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