951748 - Consistently use ‘Draft message’ instead of ‘Message draft’

Status: RESOLVED FIXED

(MailNews Core :: Account Manager, defect)

Description

[Ton [:Tonnes]]

Attachment: cc_rev13404_draft_messages.patch

After bug 562048 (TB) and bug 633937 (SM) were fixed recently, an occurrence of ‘Draft message’ passed by. There’s nothing wrong with that (on the contrary), but in the Copies & Folders section and some SM Help files, ‘Message draft’ has been used so far. This of course isn’t consistent and may confuse end users and even localizers.

http://mxr.mozilla.org/comm-central/search?string=message%2Bdraft
http://mxr.mozilla.org/comm-central/search?string=draft%2Bmessage

As ‘Draft message’ seems to be used more often by other parties and apparently is the better word when doing some searches, it would be nice to use ‘Draft message’ only. This patch changes the occurrences for both TB and SM including its Help files (where it also fixes an index typo), but leaves code like ‘messageDrafts’ as it is.

Comment 1

[:aceman]

In Copies&Folders the name "message drafts" is probably used to align with the other labels having "message archives" and "message templates". Wouldn't changing only "message drafts" appear out of place there now?

Comment 2

[Ton [:Tonnes]]

Agree, "Message draft" might have been used for a cosmetic reason (only). I wonder if that is valid one though - it wouldn't stop me from using proper terms.

Comment 3

[:aceman]

Maybe this does not need patch love, maybe Ton is still around, the patch just never asked for review.
Ton, can you refresh the patch for current trunk, if needed?

Comment 4

[Ton [:Tonnes]]

Attachment: Draft_message_v2.patch

New patch for current trunk as requested. No new or disappeared occurrences, help links tested.

Comment 5

[Richard Marti (:Paenglab)]

Comment on attachment 8751234 [details] [diff] [review]
Draft_message_v2.patch

r+ for the TB part.

Comment 6

[Philip Chee]

Comment on attachment 8751234 [details] [diff] [review]
Draft_message_v2.patch

Looks reasonable r=me

Comment 7

[:aceman]

https://hg.mozilla.org/comm-central/rev/a38deb7c45f5cc801aab3139706631dedf91b8fb