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)
Tracking
()
RESOLVED
FIXED
mozilla21
People
(Reporter: xidorn, Assigned: xidorn)
Details
Attachments
(1 file, 1 obsolete file)
1.02 KB,
patch
|
smontagu
:
review+
|
Details | Diff | 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.
Assignee | ||
Updated•12 years ago
|
Component: Untriaged → Preferences
Hardware: x86 → All
Updated•12 years ago
|
Component: Preferences → Layout: Text
Product: Firefox → Core
Updated•12 years ago
|
Attachment #697807 -
Flags: feedback?(jfkthame)
Comment 1•12 years ago
|
||
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?
Updated•12 years ago
|
Attachment #697807 -
Flags: feedback?(jfkthame)
Updated•12 years ago
|
Attachment #697807 -
Flags: feedback?(smontagu)
Comment 2•12 years ago
|
||
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.
Assignee | ||
Comment 3•12 years ago
|
||
(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.
Updated•12 years ago
|
Attachment #697807 -
Attachment is patch: true
Assignee | ||
Comment 4•12 years ago
|
||
Attachment #697807 -
Attachment is obsolete: true
Attachment #697807 -
Flags: feedback?(smontagu)
Comment 5•12 years ago
|
||
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 6•12 years ago
|
||
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+
Comment 7•12 years ago
|
||
OK, I pushed this to inbound:
https://hg.mozilla.org/integration/mozilla-inbound/rev/772f995a7e26
Assignee: nobody → quanxunzhen
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Target Milestone: --- → mozilla21
Comment 8•12 years ago
|
||
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.
Description
•