275195 - Allow reordering/customization of field order in reply header (e.g. date before author)

Status: RESOLVED WORKSFORME

(MailNews Core :: Composition, enhancement)

Description

[Jacek Maria Jackowski]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.5) Gecko/20041111 Firefox/1.0
Build Identifier: Thunderbird 1.0

When I reply I'd like to create sth like : on [date] [somebody] wrote: and also
[somebody] wrote on [Date]: ... In TB it's impossible to change this order. One
guy said me that this is a fault (I don't know - I'm not a programmer):

function -> GetReplyHeaderInfo

> static nsresult GetReplyHeaderInfo(
>     PRInt32* reply_header_type, 
>     PRUnichar** reply_header_locale,
>     PRUnichar** reply_header_authorwrote,
>     PRUnichar** reply_header_ondate,
>     PRUnichar** reply_header_separator,
>     PRUnichar** reply_header_colon,
>     PRUnichar** reply_header_originalmessage)

And the order is here
file -> mozilla\mailnews\compose\src\nsMsgCompose.cpp
file -> QuotingOutputStreamListener::QuotingOutputStreamListener

> else if (replyHeaderType == 3) 
> {
>   mCitePrefix.Append(citePrefixAuthor);
>   mCitePrefix.Append(replyHeaderSeparator);
>   mCitePrefix.Append(citePrefixDate);
> }

And because of this the order can't be changed and I think it's closing my mind
from making reply funny and imaginative

Reproducible: Couldn't Reproduce

Comment 1

[Mike Cowperthwaite]

Note that you can have the Date, Author ordering in a non-custom header by 
specifying:
  user_pref("mailnews.reply_header_type", 2);

// 0 - No Reply-Text
// 1 - "[Author] wrote:"
// 2 - "On [date] [author] wrote:"
// 3 - User-defined reply header.

// If you set 3 for the pref above then you may set the following prefs.
user_pref("mailnews.reply_header_authorwrote", "%s said");
user_pref("mailnews.reply_header_ondate", "on %s");
user_pref("mailnews.reply_header_separator", " ");
user_pref("mailnews.reply_header_colon", ":");
// The end result will be [authorwrote][separator][ondate][colon]

Comment 2

[Ivan Baldo]

I think this should be configurable from the preferences in an easy way or at
least have some predefined defaults to select from.


(In reply to comment #1)
> Note that you can have the Date, Author ordering in a non-custom header by 
> specifying:
>   user_pref("mailnews.reply_header_type", 2);
> 
> // 0 - No Reply-Text
> // 1 - "[Author] wrote:"
> // 2 - "On [date] [author] wrote:"
> // 3 - User-defined reply header.
> 
> // If you set 3 for the pref above then you may set the following prefs.
> user_pref("mailnews.reply_header_authorwrote", "%s said");
> user_pref("mailnews.reply_header_ondate", "on %s");
> user_pref("mailnews.reply_header_separator", " ");
> user_pref("mailnews.reply_header_colon", ":");
> // The end result will be [authorwrote][separator][ondate][colon]

Comment 3

[Wayne Mery (:wsmwk)]

see also bug 171822 (in fact, this should probably be duped against 171822)

Comment 4

[Hiro]

*** This bug has been marked as a duplicate of 171822 ***

Comment 5

[Mike Cowperthwaite]

No, this isn't a dupe.  171822 is about providing a UI for existing options; this bug is about providing an additional option.

Comment 6

[OstGote!]

valid RFE

Comment 7

[(not reading, please use [e-mail] instead)]

sorry for the spam.  making bugzilla reflect reality as I'm not working on these bugs.  filter on FOOBARCHEESE to remove these in bulk.

Comment 8

[:aceman]

This should be partially fixed in bug 995797 where you can change the templates for replyType=2 and replyType=3. Full featured custom format should be in bug 107884.

Comment 9

[rsx11m]

The description here only refers to the author and date information, which is fully covered by the solution in bug 995797 as template variables #1, #2, #3. Hence, this bug should be WFM?

Comment 10

[chrizilla]

(In reply to Jacek Maria Jackowski from comment #0)

When I reply I'd like to create sth like :
on [date] [somebody] wrote:
and also
[somebody] wrote on [Date]:
... In TB it's impossible to change this order.

Closing based on comment #8 and comment #9 as the reporter's wish has been implemented by bug 995797.