Fix the hidden prefs to show Sender and Message References in mail; show Message References by default in newsgroups

RESOLVED FIXED in Thunderbird 3.0b3

Status

RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: steffen.wilberg, Assigned: steffen.wilberg)

Tracking

({regression})

Trunk
Thunderbird 3.0b3
regression
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

10 years ago
The new message reader pane (bug 449691) broke the option to show message references, which was introduced by bug 62033.

Bug 62033 added the pref mailnews.headers.showReferences. When set to true, it displays the References header for emails. For newsgroup postings, the header was displayed regardless of the pref.

It's most useful if the referenced message is in another folder (like Sent/Inbox) or another newsgroup.
(Assignee)

Comment 1

10 years ago
Created attachment 366119 [details] [diff] [review]
patch

This is just a backout of the relevant part of bug 449691:
http://hg.mozilla.org/comm-central/rev/bc29303430f4#l3.1

It always shows the References header for newsgroup postings.
It's hidden by default for emails, but can be made visible by setting mailnews.headers.showReferences to true.
Attachment #366119 - Flags: ui-review?(clarkbw)
Attachment #366119 - Flags: review?(bugzilla)
I can understand wanting to add the references header back in, but why the Sender as well? You haven't explained that.
Comment on attachment 366119 [details] [diff] [review]
patch

I haven't tested yet, as waiting for explaination re the sender option. Therefore canceling review request for now, please re-request when you explain the sending option.
Attachment #366119 - Flags: review?(bugzilla)
(Assignee)

Comment 4

10 years ago
Created attachment 366663 [details] [diff] [review]
patch v1.1 - keep Sender commented out

Right, I was misled by the XXX comment and the pref.

With the first patch, the sender header would be shown whenever Sender != From. That's often the case for automated emails like like newsletters.
I don't like that extra header, so I'm commenting it out. But If we are to ditch the ability to show the sender header entirely, there's a lot more code to remove.
I'm removing the "extraExpandedHeaderList" array and the showSender pref since they're unused.

So this patch just reenables the option to show message references.
Attachment #366119 - Attachment is obsolete: true
Attachment #366663 - Flags: ui-review?(clarkbw)
Attachment #366663 - Flags: review?(bugzilla)
Attachment #366119 - Flags: ui-review?(clarkbw)
I don't think we want to ditch that option, but why are you removing it (mailnews.headers.showSender) from mailnews.js?
(Assignee)

Comment 6

10 years ago
Because the showSender pref is not used at at all:
http://mxr.mozilla.org/comm-central/search?string=showSender

If we want to make the display of the sender header configurable, we need additional code which actually checks that pref. If/when we add such code, we can reintroduce the pref as well.
(Assignee)

Comment 7

10 years ago
Created attachment 367434 [details] [diff] [review]
patch v2: fix Sender header as well

OK, this fixes the Sender header as well.

Both headers are not displayed by default; you need to set mailnews.headers.showReferences and/or mailnews.headers.showSender to true. The latter also requires a restart.
Attachment #366663 - Attachment is obsolete: true
Attachment #367434 - Flags: ui-review?(clarkbw)
Attachment #367434 - Flags: review?(bugzilla)
Attachment #366663 - Flags: ui-review?(clarkbw)
Attachment #366663 - Flags: review?(bugzilla)
Attachment #367434 - Flags: review?(bugzilla) → review+
Comment on attachment 367434 [details] [diff] [review]
patch v2: fix Sender header as well

r=me subject to Bryan's ui-review.

Bryan, (just so it is clear) this patch:

- Adds back the option via hidden pref to display the "Sender" message header in the expanded header view.

- Re-enables the references number/menu display in newsgroups so that it is always displayed (I've got a feeling this was intentionally removed?).

- Adds back the hidden preference option to display references on any email in the expanded header view.
Comment on attachment 367434 [details] [diff] [review]
patch v2: fix Sender header as well

Ok, sounds good.  Thanks for the explanation Mark!
Attachment #367434 - Flags: ui-review?(clarkbw) → ui-review+
(Assignee)

Comment 10

10 years ago
Fixed: http://hg.mozilla.org/comm-central/rev/dfb93816a016
Also tweaked the summary per comment 8.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Summary: reenable the option to show message references → Fix the hidden prefs to show Sender and Message References in mail; show Message References by default in newsgroups
(Assignee)

Updated

10 years ago
Target Milestone: --- → Thunderbird 3.0b3

Updated

9 years ago
Duplicate of this bug: 555784
You need to log in before you can comment on or make changes to this bug.