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)
SeaMonkey
MailNews: Message Display
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: bugzilla, Assigned: gerv)
References
Details
(Keywords: polish)
Attachments
(4 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
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.
Comment 1•24 years ago
|
||
moving to future milestone and reassigning to mscott.
Assignee: putterman → mscott
Target Milestone: --- → Future
Comment 2•24 years ago
|
||
Updated•24 years ago
|
Summary: Copy should be added to a-mail address context menu → Copy should be added to e-mail address context menu
Comment 3•24 years ago
|
||
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.
Updated•24 years ago
|
Updated•24 years ago
|
Severity: trivial → minor
Assignee | ||
Comment 8•24 years ago
|
||
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 → ---
Assignee | ||
Comment 9•24 years ago
|
||
Assignee | ||
Comment 10•24 years ago
|
||
Comment 11•24 years ago
|
||
Code looks okay to me, but the documentation for CopyEmailAddress is incorrect.
r=adamlock
Assignee | ||
Comment 12•24 years ago
|
||
Thanks! :-) Fixed. Now who do I go to for an sr=?
Gerv
Comment 13•24 years ago
|
||
marking nsbeta1-, but of course, since Gervase has a patch it would be cool if
this got in.
Comment 14•24 years ago
|
||
*** Bug 73616 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 15•24 years ago
|
||
Comment 16•24 years ago
|
||
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
Assignee | ||
Comment 18•24 years ago
|
||
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
Assignee | ||
Comment 19•24 years ago
|
||
Comment 20•24 years ago
|
||
sr=alecf assuming we get approval/review from a mail person
Assignee | ||
Comment 21•24 years ago
|
||
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
Comment 22•24 years ago
|
||
I'd like to see this land for 0.9 if mail-news folks are OK with it.
Whiteboard: would like for 0.9
Comment 23•24 years ago
|
||
fine with me, if that's sufficient.
Assignee | ||
Comment 24•24 years ago
|
||
It just needs someone to check it in, then. I can't, and I'm going to bed anyway
:-)
Gerv
Comment 25•24 years ago
|
||
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
Comment 26•24 years ago
|
||
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
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•