Fixed in CVS and FW_3.

I see what you mean ..



The only way I can think of doing this in a general enough way to 
allow different storage backends (ldap, sql, etc) is to change the 
changeDisplay() js function entirely.



You would have to assign an id to each entry to iterate with, then 
have the 'name' and 'address' values associated with the id.  Both 
values would be pulled in on the initial contacts.php page load, and 
change display would change the actual displayed part of the <select> 
table, instead of changing which element (value or name) is displayed. 
  Then you could conditionally show the group name instead of the 
email address if the 'address' field contained more than one \@ symbol.



Something to think about, though I realize it's a big change.  I'll 
dig into it a bit but I don't have enough familiarity with the other 
storage formats people use with turba to generalize this correctly.   
If I get a chance to make the change locally I'll post the diff for 
your inspection..



Thanks!

This won't work because the displayed strings are exactly the values 
that are sent to the compose window. Other ideas?

To reproduce:

1) open compose window

2) open address book

3) change Display type to 'email addresses'

4) if you have groups defined, this window does not wrap the addresses 
correctly, forcing user to scroll wayyyyy over to the right to see the 
'added addresses' box. The two boxes are each assigned 45% of the 
table width, but this does not seem to be respected in IE6 or Firefox 
1.03.



This may be because the entire list of addresses for the 'group' is 
being url-encoded before being put in the 'option' tag.  I suggest 
putting the Group name there instead, or truncating the list after a 
certain number of characters, with a trailing '...'.



Version Details:

Horde 3.0.4, Imp h3-4.0.3, Turba h3-2.0.2.