Closed
Bug 842324
Opened 12 years ago
Closed 12 years ago
Use "website" rather than "web site" in en-US help content
Categories
(SeaMonkey :: Help Documentation, defect)
SeaMonkey
Help Documentation
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.19
People
(Reporter: rsx11m.pub, Assigned: rsx11m.pub)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
stefanh
:
review+
|
Details | Diff | Splinter Review |
+++ 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.
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.
Comment 5•12 years ago
|
||
I'll try to take a look at this on Sunday.
Comment 7•12 years ago
|
||
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+
Updated•12 years ago
|
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]
Comment 9•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [c-n: comm-central]
Target Milestone: --- → seamonkey2.19
You need to log in
before you can comment on or make changes to this bug.
Description
•