597051 - Correct typo and adjust spacing for new Firefox compatibility UI in the HTTP preference pane

Status: RESOLVED FIXED

(SeaMonkey :: Preferences, defect)

Description

[rsx11m]

(Quoting bug 591327 comment #12)
> > I'd prefer s/Announce/Advertise/g (or whatever the U.S. spelling is)
> I agree, Advertise (or Advertize) sounds better.

Intuitively (and, more importantly, per dictionary) it is "Advertise" rather than "Advertize", thus fixing this in the following patch. Apparently the rules to translate British English to American English are somewhat inconsistent in this case.

While I'm there, I'm also including a thin separator before that group to get
a bit distance from the pipelining warning, in this way making the context of that warning more obvious. The other separators in the connection options are historically thick, I've added "thin" there as well to reduce the spacing.
This should look better overall.

Comment 1

[rsx11m]

Attachment: Simple fix [Checkin: Comment 14]

Note that I didn't rename the entity names. It's a fixed typo in en-US only without any change in semantics, thus no need to bother the localizers.

Comment 2

[rsx11m]

Attachment: Screenshots for layout comparison

From left to right:
- current spacing
- with separator added before UA string group
- separators marked "thin" in connection groups

Comment 3

[Robert Kaiser]

From all I know, the US spelling is "advertize", so there is NO typo here. Our prime locale is en-US, not en-GB. There is a separate en-GB locale for a reason.

Comment 4

[rsx11m]

I know, that's why I looked it up. That dictionary provides the en-US version, as you can verify with "analyze" (en-US) vs. "analyse" (en-GB). Also, the en-US SeaMonkey spellcheck underlines "advertize" but not "advertise", thus I'm quite confident that the proposed change is correct.

Comment 5

[Robert Kaiser]

My Langenscheidt English/German dictionary says "advertise, American also: advertize". I guess we'd need to ask American native speakers with language knowledge. I don't know if we have any of those on the SeaMonkey project.

Comment 6

[rsx11m]

I've never seen it written "advertize", which is why it caught my attention.
Adding some people with EDT time stamps to the CC list for more opinions...

Comment 7

[Philip Chee]

Typing "definition:advertize" into google gets:

Did you mean: definition:advertise  Top 2 results shown

advertise - definition of advertise by the Free Online Dictionary ...
v. ad·ver·tised, ad·ver·tis·ing, ad·ver·tis·es. v.tr. 1. To make public announcement of, especially to proclaim the qualities or advantages of (a product or ...
www.thefreedictionary.com/advertise - Cached - Similar

advertising: Definition, Synonyms from Answers.com
advertising n. The activity of attracting public attention to a product or business, as by paid announcements in the print, broadcast, or electronic.
www.answers.com/topic/advertising - Cached - Similar

Comment 8

[Robert Kaiser]

Whatever. This is not worth fighting over, and I'm far from really knowing those details well enough. Ask some Aerican or do what the reviewers decide.

Comment 9

[rsx11m]

Phil's first reference states "advertise sometimes US, advertize [ˈædvəˌtaɪz]" which would make it appear that "advertise" is preferred over "advertize" even if the latter form may be correct, but I'm flexible with that interpretation.

Comment 10

[Stefan [:stefanh]]

http://en.wikipedia.org/wiki/American_and_British_English_spelling_differences (not 100% obvious, but I could easily interpret it as 'advertise' is the right one)

Comment 11

[rsx11m]

To conclude, Stefan's Wikipedia link in comment #10 holds the key: "advertise" is not of Greek origin, thus it shouldn't be "-ize"ified in American English.

Comment 12

[Ian Neal]

Comment on attachment 475895 [details] [diff] [review]
Simple fix [Checkin: Comment 14]

Thanks for this patch r=me :)

Comment 13

[rsx11m]

Thanks for the reviews, push on comm-central please.

Comment 14

[Ian Neal]

Comment on attachment 475895 [details] [diff] [review]
Simple fix [Checkin: Comment 14]

http://hg.mozilla.org/comm-central/rev/0a06605db9b0