Right, it has to be 'remove from group', I shouldn't try to think when I'm
too tired ;)
I allowed myself to add it, I hope that was ok ... now group handling should
work as it should. :)
As a consequence of the new function, we should also change the bahavior of
the delete, shouldn't we? Actually it removes the contact only from current
group (exactly that what 'remove from group' does) with the only difference
that it deletes the contact from the last group instead of "moving" to "no
group". So I think delete should really *delete* the contact (from all
groups) and not only remove from current group.
Mirko
2007/6/2, Harry Vennik < [EMAIL PROTECTED]>:
I think you're right Mirko, users don't think of it as a virtual group. I
have it that way myself: groups for 'aMSN', 'Colleagues', 'Family',
'Friends', and anyone else, who is on my CL for some reason, in 'No group'.
So I think it might still be a good idea to allow dragging to nogroup (post
0.97 of course, so just use Phil's fix on branch).
Anyway, we need a 'remove from group' (not a 'remove from all groups', the
user can delete the contact from different groups one by one, a contact
won't be in 10 groups at the same time, I suppose).
And 'WLM behaves like that, so amsn should do, too.' is not much of an
argument in this case. We follow WLM's behavior roughly, but not always
exactly. The important thing is that WLM users that start using aMSN, should
feel very familiar with it right away.
Harry
Op 1-jun-2007, om 22:21 heeft Mirko Hansen het volgende geschreven:
Yeah, right, it's a virtual group, but the user doesn't know. He sees the
group like a normal group and I think most of them, if they use groups, try
to handle with it like that. But ok, WLM behaves like that, so amsn should
do, too. Anyway, moving back to "no group" should definitely be possible, so
we should add a "remove from all groups" to the context menu.
Mirko
2007/6/1, Youness Alaoui < [EMAIL PROTECTED] >:
>
> Indeed. Just so you know (in case you didn't realize it, but I think you
> did). The 'no group' is a virtual group
> (just like online/offline/mobile groups). It isn't listed as a group
> when we fetch the contact list. In WLM, the
> 'no group' doesn't appear, instead you have users on the top of the CL
> without appearing inside a group (this
> means you can't "collapse" the users who aren't in a group). The only
> way to put a user in 'no group' is to
> remove it from all groups. To delete a user, we remove it from all
> groups, it becomes part of 'no group' then we
> have to remove it from the 'no group' virtual group for the user to be
> deleted from our CL.
> Humm.. I just tried in WLM and it seems they changed it a bit, now users
> in 'no group' aren't appearing in the
> root of the CL, they actually appear in a "Other contacts" group under
> all other groups (instead of being on
> the top like before). If you have the option 'group contacts offline'
> (our hybrid mode), then the 'other
> contacts' group is under the 'offline' group. In WLM, there is no such
> thing as 'move contact to 'other
> contacts' group, and there is no such thing as 'remove from all groups',
> but there is the contextual menu :
> group options -> copy contact to -> group1
> group1
> etc..
> move contact to -> group1
> group2
> etc..
> remove contact from group
> The 'group options' don't appear if you sort by status.
> The 'remove contact from group' removes the user from the current group
> (in which you right clicked the
> contact).
> The 'remove contact from group' doesn't appear if the user is in 'other
> contacts'.
> If you have a user in multiple groups you have to "remove contact from
> group" from all the groups in order for
> it to go to 'other contacts'.
> As much as you can't drop a user to the 'offline' group, you also can't
> drop a user to the 'other contacts' (so
> you can't drag/drop a contact to 'no group' which would remove him from
> all other groups).
>
> That being said, I don't know what the solution with amsn should be.
> Right now, if we drop a user to 'no group',
> we get a bug report (because we send the ADC moving the contact to group
> '0' (which is no group) and the server
> responds with a 403 error, which we don't check (we do a [lindex
> $response X] to get the group id to which the
> user got moved to, and then we check the variable list_$group or
> something, so if $group is empty, we get an
> error because $list_ doesn't exist. we should check if [lindex $response
> 0] is ADC or an error message to make
> sure we don't 'handle' the ADC in the wrong way.
>
> I think dropping should be disabled for no group (as well as mobile and
> offline) and maybe just allow the
> 'delete from group' instead of only 'delete'. But I don't see why a user
> would want to put a contact in no
> group, if he wants, he can create a group named "other contacts" or "no
> group" or whatever, and move the contact
> there. the 'no group' should be used ONLY when adding a new user, then
> once you move it to a proper group, it
> should stay in a 'valid' group. That's the purpose of 'no group' in my
> opinion. I might be wrong though.
>
> So.. solution would be to disable drag/drop to no group, and leave the
> behavior as it is, if we want to allow
> users to move to no group, then the 'delete from group' method used by
> WLM would be the way to go.
> What do you think ?
>
> p.s. : sorry for the YAHEFY (yet another huge email from Youness :p).
>
> kaKaRoTo
>
>
>
> On Fri, Jun 01, 2007 at 01:31:36PM +0200, Harry Vennik wrote:
> > I think the second way is the best. However, when a contact is
> > dragged to No Group, and actually the contact is in multiple groups,
> > then the user should be warned that moving a contact to No Group will
> > remove it from any group it was in, and not just the group it was
> > dragged from.
> >
> > Harry
> >
> > Op 1-jun-2007, om 13:19 heeft Mirko Hansen het volgende geschreven:
> >
> > >Hi guys,
> > >
> > >I tried to fix the bug reported on http://www.amsn-project.net/
> > >forums/viewtopic.php?t=3418 and there I noticed that the group
> > >handling is some kind of incomplete relating especially to the "no
> > >group". To the user this group appears a normal group, although it
> > >isn't, but the user doesn't even know. So I think the user should
> > >be able to handle with this special group almost the same as with a
> > >normal group. Drag & drop to this group is/was buggy (did it ever
> > >work?), but I think it should work. WLM doesn't allow this, there
> > >it's only possible to remove a contact from all groups by using the
> > >right-click menu. So we have two possibilities: 1. behave the same
> > >way as WLM, not allowing to drop the contact on "no group" or 2. we
> > >allow it and remove the contact from all other groups if it is
> > >dropped on the "no group". I think the easiest and for n00bs most
> > >logical way is 2, even if it might be a little bit strange that
> > >copies of the contact in other groups will be removed, too, if you
> > >drop the contact in the "no group". And as a logical consequence I
> > >would add the possibility to move a contact to "no group" from the
> > >context menu. What do you think?
> > >The other way, if you think the way WLM handles this, is better, we
> > >should add something like the "remove contact from all groups" to
> > >the right-click menu, otherwise the user has no easy way to move a
> > >contact from a group to the "no group".
> > >
> > >Bye,
> > >Mirko
> >
> >----------------------------------------------------------------------
> > >---
> > >This SF.net email is sponsored by DB2 Express
> > >Download DB2 Express C - the FREE version of DB2 express and take
> > >control of your XML. No limits. Just data. Click to get it now.
> > > http://sourceforge.net/powerbar/db2/
> > >_______________________________________________
> > >Amsn-devel mailing list
> > > Amsn-devel@lists.sourceforge.net
> > > https://lists.sourceforge.net/lists/listinfo/amsn-devel
> >
>
> >
> -------------------------------------------------------------------------
> > This SF.net email is sponsored by DB2 Express
> > Download DB2 Express C - the FREE version of DB2 express and take
> > control of your XML. No limits. Just data. Click to get it now.
> > http://sourceforge.net/powerbar/db2/
> > _______________________________________________
> > Amsn-devel mailing list
> > Amsn-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/amsn-devel
>
>
> -------------------------------------------------------------------------
>
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> Amsn-devel mailing list
> Amsn-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/amsn-devel
>
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/_______________________________________________
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel
