73051 - Syntax cleanup for the Account Wizard.

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Account Configuration, defect)

Description

[[e-mail] (gone - use [e-mail] instead)]

Build ID: ALL, today's.

Just general cleanup involving parenthetical statements and where periods should
lie.  Patch next.

Comment 1

[[e-mail] (gone - use [e-mail] instead)]

Attachment: The patch.

Comment 2

[[e-mail] (gone - use [e-mail] instead)]

Attachment: New, with changes (this is representative of the changes in other areas)

Comment 3

[[e-mail] (gone - use [e-mail] instead)]

Attachment: What we used to look like.

Comment 4

[[e-mail] (gone - use [e-mail] instead)]

Note: with this new patch, because the parenthetical statement is now part of a
regular sentence, the spacing becomes a little bit off in the default size of
the window (see screenshots).  If you resize the window, it's not a problem.  

Comment 5

[[e-mail] (gone - use [e-mail] instead)]

Robin, this is perhaps hard to see in a patch only.  If you would like to come
review it on my build here, I'd be happy to show it to you.  Thanks.

Comment 6

[robinf]

r=robinf (reviewed the fixes in Stephen's debug build on his machine).

Comment 7

[[e-mail] (gone - use [e-mail] instead)]

Seth, can you sr= this please? Thanks.

Comment 8

[Hixie (not reading bugmail)]

stephend: while you're at it, can you change the sample URIs in the text to
point to one of either "example.org", "example.net", "example.com", or, if
applicable, "domain.invalid"?

In particular, emailExample.label and the comment before it should be changed 
so that the e-mail address is "[e-mail]", incomingServerNameDesc.label 
and its comment should be changed so that the POP server is "pop.example.net",
newsServerNameDesc.label and the comment before it should be changed to read
"news.example.net", and smtpServerDesc.label and its comment should have the
server name changed to "smtp.example.net".

See RFC2606 and bug 71121 for details.

Comment 9

[[e-mail] (gone - use [e-mail] instead)]

Attachment: Hixie, please review this patch.

Comment 10

[Hixie (not reading bugmail)]

You missed a comment ("<!-- LOCALIZATION NOTE (incomingServerNameDesc.label)"), 
otherwise looks ok. I don't know anything about XUL DTDs or .properties files
though, so I'm not really qualified to give an r=. Thanks!

Comment 11

[[e-mail] (gone - use [e-mail] instead)]

Attachment: Patch, changes the localization note.

Comment 12

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

sr=shaver.

Comment 13

[[e-mail] (gone - use [e-mail] instead)]

Fix checked in.  Thanks to Robin, Ian, and Mike for speedy reviews.

Comment 14

[Ninoschka Baca]

Build 2001-04-03-04: NT4
Build 2001-04-03-05: Linux RH 6.2
Build 2001-04-02-08: Mac 9.04
Verified Fixed.