249231 - Use only "web site" and "web page" in Firefox

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Jeff Walden (gone starting June 8)]

Several weeks ago a checkin (I believe only to trunk) switched Seamonkey over to
"website" from "web site" (see bug at referenced UI).  I thought Firefox only
used "Web site", as that's what I'd seen in Options and other places.  However,
I just noticed the popup notification icon tooltip has "website" in it.  A few
text searches show that there's some split in usage (even within a dialog in one
instance), as evidenced by a small sample of links:

"web site":
http://lxr.mozilla.org/aviarybranch/source/browser/locales/en-US/chrome/browser/cookieviewer/CookieExceptions.dtd#7
http://lxr.mozilla.org/aviarybranch/source/browser/locales/en-US/chrome/browser/pref/pref-advanced.dtd#27
http://lxr.mozilla.org/aviarybranch/source/browser/locales/en-US/chrome/browser/pref/pref-advanced.dtd#91

"Web site":
http://lxr.mozilla.org/aviarybranch/source/browser/locales/en-US/chrome/browser/pref/pref-features.dtd#12
http://lxr.mozilla.org/aviarybranch/source/browser/locales/en-US/chrome/browser/pageReportFirstTime.dtd#1
All of Firefox Help (standardized per my understanding that "Web site" was the
consistent choice)

"website":
http://lxr.mozilla.org/aviarybranch/source/browser/locales/en-US/chrome/browser/browser.dtd#328
http://lxr.mozilla.org/aviarybranch/source/browser/locales/en-US/chrome/browser/pref/pref-advanced.dtd#8
http://lxr.mozilla.org/aviarybranch/source/toolkit/components/passwordmgr/resources/locale/passwordmgr.properties#41
http://lxr.mozilla.org/aviarybranch/source/browser/locales/en-US/chrome/browser/cookieviewer/CookieExceptions.dtd#8

Firefox needs to standardize on one of "web site", "Web site", or "website". 
It's (very mildly) confusing, and there's no reason not to make everything look
the same.  I prefer "website" as it's vastly more common according to a Google
search for each.  (Interestingly, "website" is actually recommended as a
misspelling of "web site".)

All I need by way of response to this bug is a response from devs as to which
(website/Web site/web site) is the one wanted.  At that point I'll assign the
bug to myself and start on a patch.  I can and will make patches for the trunk
and the aviary branch with the required changes, which should be a simple (if
possibly lengthy) task.  I don't have the time to do anything until after July,
but I can and will get it done before 1.0 because it's a niggly polish issue.

Comment 1

[Jeff Walden (gone starting June 8)]

I'm requesting blocking-aviary1.0? to get a response to my question.  If I get a
response early enough, I will create a patch, so this *will* be fixed by 1.0.

Also, in the first paragraph of my last comment, it should be "see bug at
referenced URI".  Oops. ;-)

Comment 2

[Steffen Wilberg]

"web page(s)" / "Web page(s)" is a similar problem.

Comment 3

[Blake Ross]

"web site" sounds good.

Comment 4

[Jeff Walden (gone starting June 8)]

Well, I was definitely hoping more for "website" as uncapitalized and
single-word seems to be the dominant form now, but as long as it's standardized
I guess I'm okay with it.  Patch coming up (which also addresses the similar
situation with "webpage")...

Comment 5

[Jeff Walden (gone starting June 8)]

Attachment: Convert website/page to two-word, no-caps usage - should be complete

A series of greps has gotten this as close to complete as I can determine it. 
The new usage as defined by the patch is "web site" and "web page".

Comment 6

[Jeff Walden (gone starting June 8)]

Comment on attachment 155340 [details] [diff] [review]
Convert website/page to two-word, no-caps usage - should be complete

Requesting review from Blake on this...

Let me know if I should pass this review to someone else, because I'd really
like to get it in before the localization freeze (or else it won't get in until
after 1.0).

Comment 7

[Jeff Walden (gone starting June 8)]

Attachment: Same as previous with extraneous changes removed

The last patch had an extra set of changes for another bug.  I'll be switching
the review in a sec...

Comment 8

[R.J. Keller]

Comment on attachment 155345 [details] [diff] [review]
Same as previous with extraneous changes removed

r=[e-mail] for the help content. Ben, can you review this for 1.0rc1?

Comment 9

[R.J. Keller]

Ben, this is a locale change and the locale freeze is 1.0 RC 1. Since there is a
patch, I recommend reconsidering this.

Comment 10

[Jeff Walden (gone starting June 8)]

Attachment: Patch updated to latest aviary branch cvs

The previous patch (attachment 155345 [details] [diff] [review]) now generates a bunch of prompts when
applied to a tree.  This patch is updated so that it'll apply without requiring
answers to any prompts.  Nothing in Help code has been changed in this patch
that wasn't changed in the previous one, so rlk's r+ still applies to the Help
stuff.

Comment 11

[Jeff Walden (gone starting June 8)]

Comment on attachment 156001 [details] [diff] [review]
Patch updated to latest aviary branch cvs

Last I heard, blake was working through the UI text review bug list, and ben
always seems busy.  Hence, requesting review from blake...

Comment 12

[Jeff Walden (gone starting June 8)]

Comment on attachment 155345 [details] [diff] [review]
Same as previous with extraneous changes removed

Removing review request from the patch that's now prompt-hindered when applied
to aviary...

Comment 13

[chris hofmann]

lets get reviews and checked in soon if all looks well..

Comment 14

[R.J. Keller]

(In reply to comment #11)
> (From update of attachment 156001 [details] [diff] [review])
> Last I heard, blake was working through the UI text review bug list, and ben
> always seems busy.  Hence, requesting review from blake...
> 

from my experience, ben is faster than blake. But blake does seem more active in
Firefox than previously so I could be wrong.

Comment 15

[Steffen Wilberg]

I think we should at least check in the Help part, because we've changed that in
other places in Help.

Comment 16

[Jeff Walden (gone starting June 8)]

Attachment: Updated again

A few files were removed that the patch changed, so the patch would have
failed.  This patch (unlike the last few patches) also has enough context to be
fully reversible.  Will try Ben for review this time...I do so love the review
lottery.

Comment 17

[Ryan Polk (Quark)]

*** Bug 255762 has been marked as a duplicate of this bug. ***

Comment 18

[Mike Connor [:mconnor]]

Comment on attachment 156219 [details] [diff] [review]
Updated again

looks good, lets get this driven in

Comment 19

[chris hofmann]

Comment on attachment 156219 [details] [diff] [review]
Updated again

a=chofmann for the aviary branch.

Comment 20

[R.J. Keller]

Fix checked in. Just got CVS working again.

Comment 21

[Jeremy M. Dolan]

I suppose I'm a bit late weighing in, but: Because "Web" is the short form of
"World Wide Web", a proper noun, standard English dictates the W be capitalized
in "the Web", "Web site", "Web page", etc.

As Bill Walsh recently wrote, "A web site is where flies go to die".