1139004 - Rename all "Character Encoding" to "Text Encoding" in Help content (Toolkit bug 951695)

Status: RESOLVED FIXED

(SeaMonkey :: Help Documentation, defect)

Description

[rsx11m]

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

> Chrome and Internet Explorer name this entry "Encoding". Alternatively we
> could align with Safari which names it "Text Encoding".

Toolkit has changed to "Text Encoding" in toolkit/locales/en-US/chrome/global/charsetMenu.dtd which breaks suite/common/charsetOverlay.xul and thus at least needs to be changed here. Either we fork charsetMenu.dtd and its two entities to retain the old labels, or have to modify the workding throughout preference panes and Help content.

Comment 1

[rsx11m]

Since bug 1138964 covers the bustage in suite/ as well already (thanks to Paenglab!), I'm retargeting this bug to make the respective changes in the Help content.

Comment 2

[rsx11m]

(In reply to [e-mail] from bug 1138964 comment #10)
> (There's also the Page Info change...)

Neil, comm-central changeset ecc95ada09d1 didn't address generalEncoding and formEncoding in suite/locales/en-US/chrome/browser/pageInfo.dtd, thus I'll change those as drive-by fix here (unless there is a reason not to change those labels).

Comment 3

[rsx11m]

Never mind, I've filed bug 1140204 for that.

Comment 4

[rsx11m]

Attachment: Proposed patch

Note that I'm also changing the link anchors, thus had to modify mailPrefsOverlay.xul respectively.

Comment 5

[rsx11m]

Thanks Ian, push for comm-central please.

Comment 6

[Philip Chee]

http://hg.mozilla.org/comm-central/rev/3e9818b98a34

Comment 7

[Ian Neal]

Comment on attachment 8577661 [details] [diff] [review]
Proposed patch

>+++ b/suite/locales/en-US/chrome/common/help/nav_help.xhtml

>+<p>A text encoding method is the way how characters in a document or message
Sorry, only just spotted this but it should either be "the way" or "how" but not both.

>+  are converted to data to be used by your computer. All web documents and
>+  mail or news messages use a text encoding method (also known as a character
>+  set, character coding, or charset).</p>

Comment 8

[rsx11m]

Attachment: Fix wording from changeset 3e9818b98a34

Removed the "how" and rewrapped the paragraph.

Comment 9

[Ian Neal]

Comment on attachment 8632551 [details] [diff] [review]
Fix wording from changeset 3e9818b98a34

Thanks, a=me for CLOSED TREE too

Comment 10

[Philip Chee]

Comment on attachment 8632551 [details] [diff] [review]
Fix wording from changeset 3e9818b98a34

comm-central 2.40
http://hg.mozilla.org/comm-central/rev/9d4beec16911
comm-aurora 2.39
http://hg.mozilla.org/releases/comm-aurora/rev/7984ddaf8ace
comm-beta 2.38
http://hg.mozilla.org/releases/comm-beta/rev/90114be39eba
comm-release 2.37
http://hg.mozilla.org/releases/comm-release/rev/7cc9cce1a144

Comment 11

[rsx11m]

Hmm, the follow-up patch checked in long ago, thus I'm not sure why this bug is still open.
Closing as it's definitely fixed on all versions.