Closed
Bug 1130387
Opened 10 years ago
Closed 7 years ago
Fira Sans & monospace font don't handle combined diacritics correctly
Categories
(Firefox OS Graveyard :: Gaia::System, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: janjongboom, Unassigned)
References
Details
(Keywords: testcase)
Attachments
(3 files)
Open f.e. SMS app:
- Execute: document.querySelector('#messages-input').textContent = 'Hello ɛ̂ world'
- Output: Hello ɛ^world
The ^ thing is now above the next char, not on the right one.
Comment 1•10 years ago
|
||
This is actually Fira Sans (but yeah, I reproduce in Firefox using the font).
Hey Stephany, do you know who we should ping for font issues?
Flags: needinfo?(firefoxos-ux-bugzilla)
Summary: Open Sans & monospace font don't handle combined diacritics correct → Fira Sans & monospace font don't handle combined diacritics correct
Comment 2•10 years ago
|
||
Comment 3•10 years ago
|
||
Comment 4•10 years ago
|
||
Safari handles this correctly. I see: Hello ɛ̂ world.
Chrome behaves like Firefox. So it seems like a Gecko bug to me.
Comment 5•10 years ago
|
||
Somebody pointed to me that a screenshot would be more reliable.
Comment 6•10 years ago
|
||
I think this could be an edge case in Fira that exhibits a difference in behaviours. So maybe we need to fix Fira and Gecko.
Jonathan: Do you have an idea what's happening here?
Flags: needinfo?(jfkthame)
Comment 7•10 years ago
|
||
Fira Sans doesn't include the ɛ character, so font fallback picks something else; it generally won't look very good, and the diacritic won't combine properly because it will still be rendered using Fira.
The Safari screenshot looks better because it happens to pick a font (Helvetica?) that is visually quite close to Fira in weight; and I think it applies fallback to the entire base+diacritic cluster (note that the circumflex accent in Fira is rather wider and flatter than what appears in the Safari screenshot).
The fix here is to fill out the character repertoire in Fira, so that we don't hit fallback on occasional letters within the text. This should be part of completing the extended-Latin-script coverage in the fonts. cc'ing Patryk, as our liaison with the font designers.
Flags: needinfo?(jfkthame)
Comment 8•10 years ago
|
||
We would see a somewhat better result here if we completed bug 543200, but that's potentially tricky to make both fully correct and performant, given the possibilities of both precomposed and partially or fully decomposed base+diacritic combinations.
As an interim measure to mitigate this, perhaps we could do a lightweight modification to font matching, such that we try to avoid font changes between a base character and its diacritics. So if the base character hits font fallback, we'll try to use the same font for any following accent. (Though if the fallback font used for the base character doesn't support the accent needed, we'd still lose.)
I've filed bug 1130429 for that possible platform enhancement, though it doesn't remove the need to complete the character coverage of the Fira fonts so as to avoid fallback issues altogether.
Comment 9•10 years ago
|
||
Flagging Patryk on fonts.
Flags: needinfo?(firefoxos-ux-bugzilla) → needinfo?(padamczyk)
Comment 10•10 years ago
|
||
This character is being added to Fira in the next update which is slated to be released in the next few weeks.
Flags: needinfo?(padamczyk)
Updated•10 years ago
|
Summary: Fira Sans & monospace font don't handle combined diacritics correct → Fira Sans & monospace font don't handle combined diacritics correctly
Reporter | ||
Comment 11•10 years ago
|
||
[Blocking Requested - why for this release]:
This blocks bug 1128905, which has blocking-b2g 2.0+...
Blocks: 1128905
blocking-b2g: --- → 2.0?
Comment 12•10 years ago
|
||
Hey Jan, it looks like bug 1128905 covers 2 issues: one is the autocorrect issue (the original issue the bug was filed for), the other is the keyboard display issue (the issue you found while investigating).
So I object this blocks bug 1128905. We clearly have 2 different bugs. This could still be a blocker, but not for this reason.
Comment 13•10 years ago
|
||
[Triage]
de-tag given current 2.0 phase, and the symptom.
blocking-b2g: 2.0? → ---
Comment 14•7 years ago
|
||
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•