Last Comment Bug 789724 - Rename "Other Languages" to "Unicode" on the Fonts pref pane (port TB bug 323747)
: Rename "Other Languages" to "Unicode" on the Fonts pref pane (port TB bug 323...
Status: RESOLVED FIXED
:
Product: SeaMonkey
Classification: Client Software
Component: Preferences (show other bugs)
: Trunk
: All All
: -- enhancement (vote)
: seamonkey2.15
Assigned To: Jens Hatlak (:InvisibleSmiley)
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-09-08 11:44 PDT by Jens Hatlak (:InvisibleSmiley)
Modified: 2012-09-24 17:52 PDT (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
patch [Checkin: Comment 8] (3.94 KB, patch)
2012-09-08 15:07 PDT, Jens Hatlak (:InvisibleSmiley)
iann_bugzilla: review+
neil: feedback+
Details | Diff | Review

Description Jens Hatlak (:InvisibleSmiley) 2012-09-08 11:44:57 PDT
Bug 323747 renamed "Other Languages" to "Unicode" in the "Fonts & Encodings" dialog for TB. I think we should follow suit(e).

It's not very clear what "other languages" means. Technically it evaluates to Unicode. Some localizations, e.g. German, already show "Unicode" instead and I think it makes sense since it describes exactly what it means instead of introducing a term that needs explanation.

When fixing this, we should fix/adapt (en-US) Help in one go.
Comment 1 Jens Hatlak (:InvisibleSmiley) 2012-09-08 15:05:15 PDT
Overview of how this is translated into other languages (heh):
http://mxr.mozilla.org/l10n-central/search?string=font.langGroup.unicode&find=pref-fonts.dtd
Comment 2 Jens Hatlak (:InvisibleSmiley) 2012-09-08 15:07:47 PDT
Created attachment 659523 [details] [diff] [review]
patch [Checkin: Comment 8]
Comment 3 Philip Chee 2012-09-08 23:03:37 PDT
> http://mxr.mozilla.org/l10n-central/search?string=font.langGroup.unicode&find=pref-fonts.dtd

In my opinion, some of these translations already use "Unicode". Changing the entity only makes more work for those localisers. Perhaps a post to m.d.a.l10n instead?
Comment 4 Ian Neal 2012-09-09 00:42:53 PDT
Comment on attachment 659523 [details] [diff] [review]
patch [Checkin: Comment 8]

Philip's suggestion is worth considering, kairo probably knows more about l10n than me, r=me either way
Comment 5 neil@parkwaycc.co.uk 2012-09-09 02:23:29 PDT
Comment on attachment 659523 [details] [diff] [review]
patch [Checkin: Comment 8]

I notice that some locales have even translated the Firefox string as Unicode too. (Simply remove the pref- prefix to see strings for all applications.)
Comment 6 Philip Chee 2012-09-21 02:52:41 PDT
I sent a message to mozilla.dev.L10n. Let's see what they think.
Comment 7 Jens Hatlak (:InvisibleSmiley) 2012-09-24 14:17:40 PDT
(In reply to Philip Chee from comment #6)
> I sent a message to mozilla.dev.L10n. Let's see what they think.

FTR: There was just one reply, suggesting to change the entity name so we make sure that nobody misses the modification. [I'm leaving out the translation statistics provided there because they were for browser/ instead of suite/.] I tend to agree with that (better safe than sorry), so I'll go with my patch as-is.
Comment 8 Jens Hatlak (:InvisibleSmiley) 2012-09-24 14:34:53 PDT
Comment on attachment 659523 [details] [diff] [review]
patch [Checkin: Comment 8]

http://hg.mozilla.org/comm-central/rev/c652d0ae41a7
Comment 9 Robert Kaiser (not working on stability any more) 2012-09-24 17:52:47 PDT
(In reply to Jens Hatlak (:InvisibleSmiley) from comment #7)
> FTR: There was just one reply, suggesting to change the entity name so we
> make sure that nobody misses the modification. [I'm leaving out the
> translation statistics provided there because they were for browser/ instead
> of suite/.] I tend to agree with that (better safe than sorry), so I'll go
> with my patch as-is.

Sorry I didn't get to look at it, but my first reaction was the same, I feel better with doing the change of entity name so people actually do look at it.

Note You need to log in before you can comment on or make changes to this bug.