Closed Bug 826610 Opened 12 years ago Closed 12 years ago

Wrong default monospace font on Firefox for Mac with zh-CN locale

Categories

(Core :: Layout: Text and Fonts, defect)

17 Branch
All
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla21

People

(Reporter: xidorn, Assigned: xidorn)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch A patch to solve this bug. (obsolete) — Splinter Review
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:17.0) Gecko/20100101 Firefox/17.0 Build ID: 20121128204232 Steps to reproduce: Install Simplified Chinese version of Firefox for Mac. Actual results: The default monospace font is STHeiti which is not a monospace font at all. Expected results: The default monospace font should be a monospace font anyway. It is better to use the default monospace font of Mac OS X which is Menlo.
Component: Untriaged → Preferences
Hardware: x86 → All
Component: Preferences → Layout: Text
Product: Firefox → Core
Attachment #697807 - Flags: feedback?(jfkthame)
Implementation of the "monospace" font preference for non-Latin-script locales is pretty variable (as can be seen by scanning down all the "font.*.monospace.*" settings in all.js). In some cases, the default "monospace" font is the same as the default "serif" and/or "sans-serif" (also a distinction that doesn't make sense across all scripts/locales), and in some cases it's virtually impossible to have a reasonable-looking type design that is in fact monospaced. Having said that, I think it probably makes sense to make a change here, so that *Latin-script* text viewed in the zh-CN locale will use a more appropriate default for monospaced content (<pre>, etc). Whether Menlo is the right choice is a different question, though; I note that the zh-TW and zh-HK locales go for Courier, which also matches the Western pref. So maybe that would be better, in the interests of consistency. smontagu, any opinion on this? Who should be making the decision here?
Attachment #697807 - Flags: feedback?(jfkthame)
Attachment #697807 - Flags: feedback?(smontagu)
Comment on attachment 697807 [details] [diff] [review] A patch to solve this bug. In any case, if we change the default to either Menlo or Courier, we should probably keep STHeiti in the name-list preference; otherwise, I'm not sure what fallback will pick for Chinese characters that occur within monospace-styled elements but aren't available in the specified default monospace font.
(In reply to Jonathan Kew (:jfkthame) from comment #1) > Having said that, I think it probably makes sense to make a change here, so > that *Latin-script* text viewed in the zh-CN locale will use a more > appropriate default for monospaced content (<pre>, etc). Whether Menlo is > the right choice is a different question, though; I note that the zh-TW and > zh-HK locales go for Courier, which also matches the Western pref. So maybe > that would be better, in the interests of consistency. I agree that Courier is the right one since I found that Safari uses it too. I'll modify the patch.
Attachment #697807 - Attachment is patch: true
Attachment #697807 - Attachment is obsolete: true
Attachment #697807 - Flags: feedback?(smontagu)
Comment on attachment 697884 [details] [diff] [review] Modified patch, which follows the default pref of zh-TW and zh-HK I think this change makes sense, but asking smontagu for review to get another set of eyes on it.
Attachment #697884 - Flags: review?(smontagu)
Comment on attachment 697884 [details] [diff] [review] Modified patch, which follows the default pref of zh-TW and zh-HK Review of attachment 697884 [details] [diff] [review]: ----------------------------------------------------------------- Yes, this seems reasonable to me.
Attachment #697884 - Flags: review?(smontagu) → review+
Assignee: nobody → quanxunzhen
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Target Milestone: --- → mozilla21
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: