Tickets :: [#6935] Link address labels (To, CC, BCC) to address book

Summary	Link address labels (To, CC, BCC) to address book
Queue	IMP
Queue Version	4.2
Type	Enhancement
State	Rejected
Priority	1. Low
Owners	Chuck Hagenbuch <chuck (at) horde (dot) org>
Requester	Thomas Jarosch <thomas (dot) jarosch (at) intra2net (dot) com>
Created	06/16/2008 (66 days ago)
Due
Updated	07/15/2008 (37 days ago)
Assigned
Resolved	07/09/2008 (43 days ago)
Attachments	horde-imp-link-addr-fields.patch
Milestone
Patch	No

07/15/2008	Thomas Jarosch	Comment #9	Reply to this comment
And now that I think of it, we've rejected this before. Reverted. If a feature request pops up again and again, then there must be a reason for it :-) Is there a way we can have both the links and the accessor keys?

07/09/2008	Chuck Hagenbuch	Comment #8 State ⇒ Rejected	Reply to this comment
And now that I think of it, we've rejected this before. Reverted.

07/09/2008	CVS Commit	Comment #7	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/imp/compose.php?r1=2.1109&r2=2.1110&ty=u http://cvs.horde.org/diff.php/imp/docs/CHANGES?r1=1.1158&r2=1.1159&ty=u

07/09/2008	Jan Schneider	Comment #6	Reply to this comment
I think they're redundant, because we have the same button up to 4 times now on the compose screen. But if people really want them, they should be separate buttons I guess.

07/09/2008	Chuck Hagenbuch	Comment #5	Reply to this comment
Would you suggest the old labels with accesskeys, and separate buttons? Or would you veto the buttons in general?

07/09/2008	Jan Schneider	Comment #4 State ⇒ Feedback	Reply to this comment
I'm really unhappy with the implementation, because now, if you use an access key to navigate to the address fields, the cursor is no longer placed into the field, but instead the address book pops up. Actually the labels don't work anymore at all, which also reduces accessibility.

06/16/2008	Chuck Hagenbuch	Comment #3 State ⇒ Resolved Assigned to Chuck Hagenbuch	Reply to this comment
Committed, thanks.

06/16/2008	CVS Commit	Comment #2	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/imp/compose.php?r1=2.1096&r2=2.1097&ty=u http://cvs.horde.org/diff.php/imp/docs/CHANGES?r1=1.1141&r2=1.1142&ty=u

06/16/2008	Thomas Jarosch	Patch ⇒ 1

06/16/2008	Thomas Jarosch	Comment #1 New Attachment: horde-imp-link-addr-fields.patch Patch ⇒ 1 Milestone ⇒ Queue ⇒ IMP Summary ⇒ Link address labels (To, CC, BCC) to address book Type ⇒ Enhancement Priority ⇒ 1. Low State ⇒ New	Reply to this comment
Hello together, attached patch links the address labels (To, CC, BCC) to the address book as it's quite common in some desktop mail clients and users are used to it. Please check if the close of the <a href> is done right by appending "</a>". Cheers, Thomas