Closed Bug 1244017 Opened 4 years ago Closed 4 years ago

Font for FULL WIDTH symbols is small in UI on OSX.

Categories

(Core :: Graphics: Text, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla47
Tracking Status
firefox46 --- fixed
firefox47 --- fixed

People

(Reporter: arai, Assigned: jtd)

References

(Blocks 1 open bug)

Details

(Keywords: regression, Whiteboard: [gfx-noted])

Attachments

(6 files)

Attached image ui-font.png
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.
it looks strange when FULL WIDTH DIGIT characters are mixed with other Japanese characters.
Whiteboard: [gfx-noted]
Attached image ui-font2.png
noticed that it's not restricted to FULL WIDTH DIGIT, but most of (all?) FULL WIDTH symbols, except alphabet.
Summary: Font for FULL WIDTH DIGIT is small in UI on OSX. → Font for FULL WIDTH symbols is small in UI on OSX.
Ack, we're picking up full-width digits from Apple Symbols. So this is clearly not the same list used by Safari.
*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: nobody → jd.bugzilla
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+
Flags: needinfo?(jd.bugzilla)
Attachment #8716833 - Flags: review?(m_kato)
Attachment #8716833 - Flags: review?(m_kato) → review+
https://hg.mozilla.org/mozilla-central/rev/149dc37ee4c7
https://hg.mozilla.org/mozilla-central/rev/ac338559876d
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Note that mozilla46 (currently Beta) is still affected by this. In bug 1240407 comment 26, I'm proposing we should uplift this backout.
Noted, and the uplift in bug 1240407 is approved for beta 46.
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.