Closed Bug 761720 Opened 9 years ago Closed 9 years ago

Port |Bug 474721 - messagereader: message header should not prefer the address book over the message headers as a source of display names|

Categories

(SeaMonkey :: MailNews: Message Display, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.13

People

(Reporter: iann_bugzilla, Assigned: iann_bugzilla)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #474721 +++

In my neck of the woods, Evite is a very widely used event invitation system, and it sends out mails with headers like this:

From: Bill Schlep <info@mail.evite.com>
Reply-To: Bill Schlep <bills_real_address@foo.com>

They also include this string in the body of the message: "Did this email go to your junk/bulk folder?  Add info@evite.com to your address book to ensure that you receive future Evite Invitations in your Inbox." which explains the motivation (though they have a bug in that they changed the hostname their mail came from without updating the body text to match).

This means that currently, all evites that I get appear, in the compact header view, as though they came from whatever name is associated with info@evite.com, which happens to be whoever I got the evite from when I starred that address.  Very confusing.

ping, an up-and-coming web 2.0 invite site does something similar.

If an email has a display in the From header and we prefer that to anything from the address book, that fixes this problem.
Blocks: 761852
Comment on attachment 630237 [details] [diff] [review]
Simple port on top of patch from bug 131571 [Checked in: Comment 3]

Just some nits:

>+        // And the "prefer display name" checkbox

"and" or full stop at the end.

>+  if (condense && cardDetails.card) {

Brace on its own line, please.


r/moa=me with that.
Attachment #630237 - Flags: superreview+
Attachment #630237 - Flags: review?(mnyromyr)
Attachment #630237 - Flags: review+
Comment on attachment 630237 [details] [diff] [review]
Simple port on top of patch from bug 131571 [Checked in: Comment 3]

Checked in with points addressed:
http://hg.mozilla.org/comm-central/rev/172686daec10
Attachment #630237 - Attachment description: Simple port on top of patch from bug 131571 → Simple port on top of patch from bug 131571 [Checked in: Comment 3]
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.13
Blocks: 763284
Blocks: 773867
"always prefer display name over message header" in addressbook is no longer working in at least 2.19, one other user in Seamonkey IRC could replicate this bug.
You need to log in before you can comment on or make changes to this bug.