Closed Bug 842324 Opened 7 years ago Closed 7 years ago

Use "website" rather than "web site" in en-US help content

Categories

(SeaMonkey :: Help Documentation, defect, trivial)

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.19

People

(Reporter: rsx11m.pub, Assigned: rsx11m.pub)

References

Details

Attachments

(1 file, 1 obsolete file)

+++ This bug was initially created as a clone of Bug #842320 +++

Following the observation in bug 835134 comment #36 (and a similar case in bug 416234 comment #9) that en-US is supposed to use "website" (rather than "web site"), I had another look at the preference panes to find any remaining instances. I only found a single one in the labels, blockAutoRefresh.label, but several occurrences in the help content.

Thus, I'll use this [bug 842320] for correcting the label and will follow up on the help content with Stefan with a separate patch.
So, in essence this is the continuation of bug 423282 catching any remaining or new items in suite/locales/en-US/chrome/common/help. I'm using the same approach as for bug 652328 part I to find those, which seems to affect a handful of XHTML files.

In addition, even for those instances where "web site" was already changed to "website" previously, identifiers and links may still show "web_site" in that part and are thus inconsistent with the actual title they represent. Those should be easy to correct.

I'll post two patches (one with just the text changes, the other one also including link changes) for you to pick from, in case you think that adjusting the links too might be an overkill.
Attached patch Patch retaining any links (obsolete) — Splinter Review
This is the patch only changing the text but leaving any anchors untouched. After bug 842820 resolving two instances, there are five "web site" instances left in three files.

Note that I couldn't find any actual label "Edit website certificate trust settings" as referred to in certs_help.xhtml, but that's outside the scope of this specific bug.
Attachment #715238 - Flags: review?(stefanh)
This is the longer patch, also globally changing all "web_site" to "website" in all links found in the *.xhtml and *.rdf files within help. I've verified that there are no references from any *.xul files to either of these instances.

Please r+ one of them (maximum), whichever version you like better.
Attachment #715241 - Flags: review?(stefanh)
(In reply to rsx11m from comment #2)
> Note that I couldn't find any actual label "Edit website certificate trust
> settings" as referred to in certs_help.xhtml, [...]

Never mind, that's hiding as certmgr.editsslcert.title entity in
security/manager/locales/en-US/chrome/pippki/certManager.dtd#42
and already has the correct "website" in its label.
I'll try to take a look at this on Sunday.
Thanks, it's hopefully just a simple up-or-down vote.
Comment on attachment 715241 [details] [diff] [review]
Patch also adjusting the links

Let's go for the longer version ;-)
Attachment #715241 - Flags: review?(stefanh) → review+
Attachment #715238 - Attachment is obsolete: true
Attachment #715238 - Flags: review?(stefanh)
Thanks, push of attachment 715241 [details] [diff] [review] for trunk please.
Keywords: checkin-needed
Whiteboard: [c-n: comm-central]
http://hg.mozilla.org/comm-central/rev/09f6cca502d1
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [c-n: comm-central]
Target Milestone: --- → seamonkey2.19
Blocks: 423282
You need to log in before you can comment on or make changes to this bug.