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

6.0.0-beta1

7/7/25

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 (at) horde (dot) org
Requester	thomas.jarosch (at) intra2net (dot) com
Created	06/16/2008 (6230 days ago)
Due
Updated	07/15/2008 (6201 days ago)
Assigned
Resolved	07/09/2008 (6207 days ago)
Milestone
Patch	No

07/15/2008 02:29:13 PM	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 08:47:30 PM	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 08:47:13 PM	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 08:19:25 PM	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 06:26:25 PM	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 12:17:21 PM	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 08:14:27 PM	Chuck Hagenbuch	Comment #3 State ⇒ Resolved Assigned to Chuck Hagenbuch	Reply to this comment
Committed, thanks.

06/16/2008 08:10:26 PM	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 02:23:51 PM	Thomas Jarosch	Patch ⇒ Yes

06/16/2008 02:10:28 PM	Thomas Jarosch	Comment #1 Priority ⇒ 1. Low State ⇒ New New Attachment: horde-imp-link-addr-fields.patch Patch ⇒ Yes Milestone ⇒ Queue ⇒ IMP Summary ⇒ Link address labels (To, CC, BCC) to address book Type ⇒ Enhancement	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