1366863 - nbsp characters are not displayed with Zilla font

Status: RESOLVED FIXED

(www.mozilla.org :: General, defect)

Description

[Théo Chevalier]

Attachment: Capture d’écran 2017-05-22 à 20.30.15.png

Non breaking spaces are missing with the new Zilla font.

They are heavily used in French for instance, I’m attaching an example from this page: https://www.mozilla.org/fr/internet-health/privacy-security/ (there should be a space before ":")

Using the Web Inspector, you can see the space is there, but it’s not rendered.

Comment 1

[Alex Gibson [:agibson]]

For bugs with the font itself it probably makes more sense to open issues on the GitHub repo. I've opened an issue here for this particular case:

https://github.com/mozilla/zilla-slab/issues/7

Comment 2

[Théo Chevalier]

Attachment: Screenshot

Thanks for opening the issue, Alex.

Adding another screenshot to provide a better overview of how bad it looks.
If there is a temporary solution to work around this issue until the font is fixed, I think it should be considered.

Comment 3

[Alex Gibson [:agibson]]

Needsinfo'ing Yulia, who can speak to the font designers. I'm not sure there's any workaround here other than updating the font.

Comment 4

[yuliya]

Which weight of the typeface is this happening in?

In the current version it should only happen in the Bold Highlight weight. It's turned off in all other weights.

Comment 5

[Craig Cook (:craigcook)]

(In reply to yuliya from comment #4)
> Which weight of the typeface is this happening in?
> 
> In the current version it should only happen in the Bold Highlight weight.
> It's turned off in all other weights.

I can reproduce this error in Zilla Slab Regular, Italic, Bold, and Bold Italic, which are the only weights currently in use on mozilla.org. I haven't tested Light or Medium but suspect they have the same flaw. The character definitely exists in the font or else it would be substituted by a space character from a fallback font. It causes problems were an explicit space character is required for correct formatting (as in French) so we'll need to get this corrected as soon as possible.

Comment 6

[yuliya]

Attachment: ZillaSlabTestFontsR01(1).zip

Craig, which version of the font are using? Attaching the latest one here. This problem should have been solved.

Comment 7

[Craig Cook (:craigcook)]

(In reply to yuliya from comment #6)
> Created attachment 8881341 [details]
> ZillaSlabTestFontsR01(1).zip
> 
> Craig, which version of the font are using? Attaching the latest one here.
> This problem should have been solved.

Mozilla.org and MDN are using the version in the github repo at https://github.com/mozilla/zilla-slab/. The version you've attached fixes the collapsing space and I've submitted pull requests to update it on github as well as on mozorg (not MDN yet but we'll get that one next).

https://github.com/mozilla/zilla-slab/pull/11
https://github.com/mozilla/bedrock/pull/4935

Comment 8

[[github robot]]

Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/1d1f554c7c0265a597cd0b2eb3480bc70798bfe3
[fix bug 1366863] Update Zilla Slab fonts

https://github.com/mozilla/bedrock/commit/8126458e6459ccd2fad08c7667c3654ec6900e7f
Merge pull request #4935 from craigcook/bug-1366863-update-zilla-slab

[fix bug 1366863] Update Zilla Slab fonts