Closed Bug 294678 Opened 20 years ago Closed 20 years ago

Change "Website" to "web site" throughout Reporter

Categories

(Other Applications Graveyard :: Reporter, defect)

Product:
Other Applications Graveyard
Component:
Reporter
Type:
defect
Priority:
Not set
Severity:
trivial

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: Waldo, Assigned: Waldo)

Details

Attachments

(2 files)

Use "web site" instead of "Website" and friends
4.52 KB, patch
raccettura
: review+
asa
: approval-aviary1.1a1+
Details | Diff | Splinter Review
Fix problems on r.m.o
1.67 KB, patch
raccettura
: review+
Details | Diff | Splinter Review 
Back before 1.0, I made an effort in bug to get all of Firefox to spell "web
site" consistently (instead of having "website", "Web site", "web site", etc.
all vying for primacy).  It was a polish issue, and since then as far as I know
we've pretty much kept to it.

Consistent applications are better applications, and it's little things like
this that make Firefox better than other applications.  Firefox now includes
Reporter, but Reporter doesn't follow this rule.  It should.

Patch coming in a sec...
Before I forget to mention it, the "web site" bug was bug 249231.

I don't remember internals enought to remember if the change in the last file
(contents.rdf) is needed or not, but I'd prefer to make it just to be
absolutely certain the problem's fixed.
Assignee: robert → jwalden+bmo
Status: NEW → ASSIGNED
Attachment #183919 - Flags: review?(robert)
Comment on attachment 183919 [details] [diff] [review]
Use "web site" instead of "Website" and friends

The text in the RDF is what prints under the extension name in the EM.

r=robert@accettura.com
Attachment #183919 - Flags: review?(robert) → review+
There's also a problem with this on r.m.o in the license page.	This patch
fixes the problem, although if this is going to be very, very substantially
rewritten then it's probably not worth the effort.

Also, note that the patch closes the <head> element in the page, which
currently isn't closed and causes at least a few validation errors.
Attachment #183920 - Flags: review?(robert)
Comment on attachment 183920 [details] [diff] [review]
Fix problems on r.m.o

Were not done here, but yea, for consistancy sake, let just make the change.

r=robert@accettura.com
Attachment #183920 - Flags: review?(robert) → review+
I'll deploy the r.m.o part in a few days.
Comment on attachment 183919 [details] [diff] [review]
Use "web site" instead of "Website" and friends

(In reply to comment #2)
> The text in the RDF is what prints under the extension name in the EM.

I'm not so sure about that - first, because if it did it would require l10n,
and second, because my trunk build doesn't show any information regarding
Reporter in the extension list (which is probably a good thing, because we
don't want people removing it).
Attachment #183919 - Flags: approval-aviary1.1a1?
Comment on attachment 183919 [details] [diff] [review]
Use "web site" instead of "Website" and friends

a=asa
Attachment #183919 - Flags: approval-aviary1.1a1? → approval-aviary1.1a1+
Comment on attachment 183919 [details] [diff] [review]
Use "web site" instead of "Website" and friends

Robert, feel like checking this in?  I'm technically limited to browser,
toolkit, and help component stuff.
Done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: