368618 - improve wording for "Sending authenticate login information..." status

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Mark Banner (:standard8)]

Attachment: Fix v1

SeaMonkey equivalent of bug 300670. I'm proposing we do the same as Thunderbird.

I'm only going to propose this for trunk as its l10n.

Comment 1

[[e-mail]]

Comment on attachment 253228 [details] [diff] [review]
Fix v1

This is the message used when you turn on secure authentication, right? In which case, shouldn't it mention the word "secure" somewhere?

Comment 2

[[e-mail]]

Comment on attachment 253228 [details] [diff] [review]
Fix v1

See bug 300670 comment #10.

Comment 3

[Serge Gautherie (:sgautherie)]

Attachment: (Av2) "secure login"

Like this ?

If you want to use "5014=Sending secure authentication...",
I believe we should use "5013=Sending authentication..." too...
and maybe update other places too: <http://mxr.mozilla.org/seamonkey/search?string=login+information>.

Comment 4

[[e-mail]]

Comment on attachment 285749 [details] [diff] [review]
(Av2) "secure login"

FYI, login and authentication are different IMAP commands, which is why it doesn't make sense to change the insecure login message.

Comment 5

[Serge Gautherie (:sgautherie)]

Attachment: (Av2b) "secure login", + empty line removals [Checkin: comment 8]

Same as Av2, plus empty line removals: sync'ing with TB.
(Sorry not to have done it the first time.)

Comment 6

[Serge Gautherie (:sgautherie)]

Comment on attachment 285867 [details] [diff] [review]
(Av2b) "secure login", + empty line removals
[Checkin: comment 8]

"approval‑seamonkey1.1.6=?":
Not mandatory; only to "synchronize" with 'mscott: approval‑thunderbird2+' from (TB) bug 300670.

Comment 7

[Robert Kaiser]

Comment on attachment 285867 [details] [diff] [review]
(Av2b) "secure login", + empty line removals
[Checkin: comment 8]

This changes a string in L10n files, which we can't permit after the L10n freeze which happened for 1.1 Beta.

Comment 8

[Mark Banner (:standard8)]

Checked in:

Checking in suite/locales/en-US/chrome/mailnews/imapMsgs.properties;
/cvsroot/mozilla/suite/locales/en-US/chrome/mailnews/imapMsgs.properties,v  <--  imapMsgs.properties
new revision: 1.48; previous revision: 1.47

Thanks for finishing this off for me :-)

Comment 9

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a9pre) Gecko/2007102402 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

V.Fixed,
checking manually <...\seamonkey\chrome\en-US.jar\locale\en-US\messenger\imapMsgs.properties>.