rename unicode langGroup to 'Other(s) languages/scripts'(?)



14 years ago
10 years ago


(Reporter: jshin1987, Assigned: jshin1987)



Firefox Tracking Flags

(Not tracked)



(2 attachments, 1 obsolete attachment)



14 years ago
This was 'inspired' by bug 280001 and many others just like that.

One of the most confusing aspects of Mozilla font-pref. UI is what 'font pref.
entries for Unicode' are for. Currently, they're only used to set fonts for
langGroups for which we don't have separate font pref. entries (e.g. Ethiopian,
Bengali, etc).(bug 256383) It's NOT for setting fonts for pages in UTF-8/16/32
*without* lang, but users often get confused by that as shown by bug 280001 and
others. [1] 

Having said that, I think we may not have to fix bug 91190. Instead, we may as
well just rename 'Unicode langGroup' to 'Other scripts/languages'. Actually, bug
91190 and renaming are rather orthogonal to each other. So, we have to do both,
but independently of each other. 

[1]  In UTF-8/16/32 pages (and to a lesser degree in pages in non-Unicode
encoding), it's always a good idea to specify 'lang' and/or 'xml:lang'. See

Comment 1

14 years ago
Created attachment 173453 [details] [diff] [review]
patch (UI name change)

 I just changed the UI name, but didn't change our internal names ('x-unicode',
'langGroup.unicode',etc). I can do that, but I don't think it's necessary
(it'll be a huge patch). What do you think?

Comment 2

14 years ago
Created attachment 173454 [details] [diff] [review]
help file patch

I added a sentence about 'Other languages'. While I'm at it, I also fixed FF
help (I thought I fixed it while working on the suite help, but apparently I
Attachment #173454 - Flags: review?(steffen.wilberg)

Comment 3

14 years ago
Comment on attachment 173454 [details] [diff] [review]
help file patch

r=me if you actually change the entity string from "Unicode" to "Other
languages" in﷒0﷓. Your first patch only changes xpfe and mail.

And please use <em> instead of <q> in Firefox Help.
Attachment #173454 - Flags: review?(steffen.wilberg) → review+

Comment 4

14 years ago
Created attachment 173455 [details] [diff] [review]
patch (UI name change) : missing browser part added

I forgot to include browser/* part in the previous patch. (thanks Steffen for
catching that.) Do I need to ask a FF peer for review as well?
Attachment #173453 - Attachment is obsolete: true
Attachment #173455 - Flags: superreview?(dbaron)
Attachment #173455 - Flags: review?(smontagu)


14 years ago
Attachment #173455 - Flags: review?(smontagu) → review+

Comment 5

14 years ago
"For codebase-wide simple, repetitive changes (such as relicensing, spelling,
whitespace or capitalization changes), review from a Firefox Peer is not
required as long as the patch as a whole has review."

This is a codebase-wide (xpfe, mail, browser), simple (change of a single
entity), and repetitive (xpfe, mail, browser) change, isn't it? ;-)

Comment 6

14 years ago
thanks for r/sr. landed on the trunk
Last Resolved: 14 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.