Closed
Bug 1244017
Opened 9 years ago
Closed 9 years ago
Font for FULL WIDTH symbols is small in UI on OSX.
Categories
(Core :: Graphics: Text, defect)
Core
Graphics: Text
Tracking
()
RESOLVED
FIXED
mozilla47
People
(Reporter: arai, Assigned: jtd)
References
(Blocks 1 open bug)
Details
(Keywords: regression, Whiteboard: [gfx-noted])
Attachments
(6 files)
After bug 1212731, font for FULL WIDTH DIGIT in UI gets small, looks like "Apple Symbols" is used, on OSX 10.10.5, with Japanese locale user setting. It was "Hiragino Kaku Gothic" (maybe) before, and Safari also uses it.
Attached screenshot of Firefox nightly with clean profile, and safari.
Reporter | ||
Comment 1•9 years ago
|
||
it looks strange when FULL WIDTH DIGIT characters are mixed with other Japanese characters.
Updated•9 years ago
|
Whiteboard: [gfx-noted]
Reporter | ||
Comment 2•9 years ago
|
||
noticed that it's not restricted to FULL WIDTH DIGIT, but most of (all?) FULL WIDTH symbols, except alphabet.
Reporter | ||
Updated•9 years ago
|
Summary: Font for FULL WIDTH DIGIT is small in UI on OSX. → Font for FULL WIDTH symbols is small in UI on OSX.
Assignee | ||
Comment 3•9 years ago
|
||
Ack, we're picking up full-width digits from Apple Symbols. So this is clearly not the same list used by Safari.
Assignee | ||
Comment 4•9 years ago
|
||
*sigh* As the system font cascade code seems to cause a startup regression of roughly 40ms and in this case it's causing regressions, I think it's best to pull this code out for now.
Attachment #8716184 -
Flags: review?(m_kato)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → jd.bugzilla
Comment 5•9 years ago
|
||
Comment on attachment 8716184 [details] [diff] [review]
patch, remove system font cascade for OSX
Review of attachment 8716184 [details] [diff] [review]:
-----------------------------------------------------------------
We should investigate why Safari/WebKit works font linking well for system font after landing this.
Attachment #8716184 -
Flags: review?(m_kato) → review+
Comment 7•9 years ago
|
||
sorry had to back this out in https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=820ab8931ebc seems this push caused https://treeherder.mozilla.org/logviewer.html#?job_id=21123844&repo=mozilla-inbound
Flags: needinfo?(jd.bugzilla)
Assignee | ||
Comment 8•9 years ago
|
||
Flags: needinfo?(jd.bugzilla)
Attachment #8716833 -
Flags: review?(m_kato)
Updated•9 years ago
|
Attachment #8716833 -
Flags: review?(m_kato) → review+
Comment 10•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/149dc37ee4c7
https://hg.mozilla.org/mozilla-central/rev/ac338559876d
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Comment 11•9 years ago
|
||
Note that mozilla46 (currently Beta) is still affected by this. In bug 1240407 comment 26, I'm proposing we should uplift this backout.
status-firefox46:
--- → affected
Comment 12•9 years ago
|
||
Noted, and the uplift in bug 1240407 is approved for beta 46.
Comment 13•9 years ago
|
||
Marking fixed on 46, as the backout in bug 1240407 has now landed there.
You need to log in
before you can comment on or make changes to this bug.
Description
•