The default bug view has changed. See this FAQ.

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

RESOLVED FIXED in seamonkey2.13

Status

SeaMonkey
MailNews: Message Display
--
enhancement
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: Ian Neal, Assigned: Ian Neal)

Tracking

(Blocks: 1 bug)

Trunk
seamonkey2.13
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
+++ 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.
(Assignee)

Comment 1

5 years ago
Created attachment 630237 [details] [diff] [review]
Simple port on top of patch from bug 131571 [Checked in: Comment 3]
Attachment #630237 - Flags: review?(mnyromyr)
(Assignee)

Updated

5 years ago
Blocks: 761852

Comment 2

5 years ago
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+
(Assignee)

Comment 3

5 years ago
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]
(Assignee)

Updated

5 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.13
(Assignee)

Updated

5 years ago
Blocks: 763284
(Assignee)

Updated

5 years ago
Blocks: 773867

Comment 4

4 years ago
"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.