Closed Bug 47743 Opened 24 years ago Closed 24 years ago

Copy should be added to e-mail address context menu

Categories

(SeaMonkey :: MailNews: Message Display, defect, P3)

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: bugzilla, Assigned: gerv)

References

Details

(Keywords: polish)

Attachments

(4 files)

When you right-click on a e-mail address in the header field of the messagepane, Copy should appear in the context menu. Currently the context menu only has: Add address to address book Send mail to The Copy would copy the email address to the clipboard.
moving to future milestone and reassigning to mscott.
Assignee: putterman → mscott
Target Milestone: --- → Future
I think this context menu should share menu items with the mailto: link context menu in bug 32358. Then we'd automatically have the same mnemonics in both menus, and internationalization teams would have fewer things to translate. See also discussion in bug 57578 about removing this context menu :(
Summary: Copy should be added to a-mail address context menu → Copy should be added to e-mail address context menu
The context menu for these addresses should be the same as for any other e-mail address link in a Web page or e-mail message -- though the default item would be different (`Add to Address Book ...' for an address in the header pane, vs. `New Message' in Web page/message content). Bug 57578 isn't about removing the context menu, it is about turning the current compulsory menu into a context menu.
nominating nsbeta1 to add copy to the menu.
Keywords: nsbeta1
QA Contact: lchiang → laurel
marking nsbeta1-
Keywords: nsbeta1nsbeta1-
Severity: normal → trivial
Keywords: polish, ui
Severity: trivial → minor
*** Bug 54894 has been marked as a duplicate of this bug. ***
*** Bug 63023 has been marked as a duplicate of this bug. ***
OK - patches coming right up. I would appreciate review of these :-) Resetting target milestone and nsbeta1 nomination for retriage... Hope they work - my editor does funny things with the tabs in the file, so there's been a little hand-hackery of the patches. Gerv
Target Milestone: Future → ---
Attached patch Patches to content files (deleted) — Splinter Review
Attached patch Patch to US DTD (deleted) — Splinter Review
Code looks okay to me, but the documentation for CopyEmailAddress is incorrect. r=adamlock
Thanks! :-) Fixed. Now who do I go to for an sr=? Gerv
Blocks: 32358
marking nsbeta1-, but of course, since Gervase has a patch it would be cool if this got in.
Keywords: nsbeta1nsbeta1-
*** Bug 73616 has been marked as a duplicate of this bug. ***
Attached patch Updated diff - looking for sr= (deleted) — Splinter Review
Oops, this bug doesn't block bug 32358, since for now the context menu for e- mail addresses in mailnews xul is different from the context menu for e-mail addresses in html content.
No longer blocks: 32358
r=timeless
Assignee: mscott → gervase.markham
sspitzer - any chance of a sr= here? :-) [ Sidenote: you and dveditz get the prize for the people whose name is most often misspelt in checkin comments...] Gerv
Depends on: 75338
sr=alecf assuming we get approval/review from a mail person
sspitzer, mscott or bienvenu: This bug has r= and sr=, it just needs a "Yes, we'd like that" from someone in mail/news. Can that be provided? I know Asa wants this really badly ;-) Gerv
I'd like to see this land for 0.9 if mail-news folks are OK with it.
Whiteboard: would like for 0.9
fine with me, if that's sufficient.
It just needs someone to check it in, then. I can't, and I'm going to bed anyway :-) Gerv
fix by Gervase Markham mailto:gervase.markham@univ.ox.ac.uk checked in. Sorry gerv i messed up the cvs comment. If anyone has any problems with this bug please open a new one against gerv or me cc the other and cite this bug. I'm very sorry.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Whiteboard: would like for 0.9
OK using apr23 commercial trunk build. Win98, linux rh6.2, mac OS 9.0 Context menu item present and functional from mail and news header envelopes. Any other issues with this item will be logged separately.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: