1150276 - Ship Fira and Charis SIL with Reader View for know character supported locales

Status: RESOLVED WONTFIX

(Toolkit :: Reader Mode, defect, P3)

Description

[Michael Maslaney [:mmaslaney] ([e-mail])]

Ship Fira and Charis with Reader View for know supported locales.

SPECS

Charis Spec: http://invis.io/4V2K8JN9E

Fira Spec: http://invis.io/JH26PPWT5


FONTS & WEIGHTS TO INCLUDE:

Charis SIL – Regular, Bold, Italic, Bold Italic

Fira – Book, Bold, Book Italic, Bold Italic

Fira Monotype – Regular


FIRA Language SUPPORT: 

• English
• Spanish
• German 
• French 
• Russian 
• Dutch 
• Portuguese 
• Italian
• Greek
• Turkish
• Swedish
• Polish
• Swedish
• Danish
• Czech
• Indonesian
• Filipino
• Norwegian
http://www.carrois.com/en/fira-3-1/


CHARIS Language Support

http://scripts.sil.org/cms/scripts/page.php?site_id=nrsi&id=CharisSIL_Technical#38fec032


CSS:

font-family: "Charis SIL",Georgia,"Times New Roman",serif;

font-family: "Fira Sans",Helvetica,Arial,sans-serif

Comment 1

[Michael Maslaney [:mmaslaney] ([e-mail])]

Updated Fira support:
http://www.carrois.com/en/fira-4-1/

Comment 2

[Michael Maslaney [:mmaslaney] ([e-mail])]

Attachment: FiraSans-Book.pdf

Comment 3

[Blake Winton (:bwinton) (:☕️)]

Attachment: Attempt number 1.

So, how does this look as a way of getting locale-dependant font choices?
Is there a better way of doing this?

(I still need to adjust the font-sizing buttons, so this isn't ready for review, but if the approach looks good and there's stuff I should be fixing, I would be happy to fix that now, too.)

Comment 4

[Blake Winton (:bwinton) (:☕️)]

(I think this might be tricky.)

Comment 5

[Blake Winton (:bwinton) (:☕️)]

Comment on attachment 8598108 [details] [diff] [review]
Attempt number 1.

Screenshots for UI-Review.

Mac Retina: https://www.dropbox.com/s/vzv0cmu04jifj79/Screenshot%202015-04-27%2013.54.04.png?dl=0

Windows @200%: https://www.dropbox.com/s/64mlyie7ezlqlg0/Screenshot%202015-04-27%2015.28.34.png?dl=0
Windows @100%: https://www.dropbox.com/s/v001mu4larremki/Screenshot%202015-04-27%2015.30.02.png?dl=0

Linux: https://www.dropbox.com/s/j5qgjevnxkd604m/Screenshot%202015-04-27%2016.39.16.png?dl=0

Comment 6

[:Gijs (he/him)]

Comment on attachment 8598108 [details] [diff] [review]
Attempt number 1.

Review of attachment 8598108 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks for writing a patch!

Sadly, I foresee problems with this:

0) the browser locale != the text on the page. Basically, if I'm using a locale where these fonts don't work, but view an English article, wouldn't I want the other font? Or would it be confusing? What if I used en-US but read an article in a language where the font doesn't work?
1) While our localizers are awesome, I don't know how many will be able to make good choices for these values for their locale anyway
2) I don't think our l10n infrastructure is equipped for localized CSS files, unfortunately. :Pike would know for sure.

Comment 7

[Blake Winton (:bwinton) (:☕️)]

Pike, can you give me any ideas on how to handle localizing CSS files (if such a thing is possible or desirable)?

Comment 8

[Axel Hecht [:Pike]]

I'm a bit afk-pto, but quickly:

You don't care about UI locale, but content locale. Thus you should use the language detector logic we had for in-page-translation, and based on that language, insert CSS into the reader-view page to either use the shipped fonts, or default OS fonts.

Comment 9

[Dão Gottwald [:dao]]

Comment on attachment 8598108 [details] [diff] [review]
Attempt number 1.

(In reply to :Gijs Kruitbosch (mostly away until May 5th) from comment #6)
> 0) the browser locale != the text on the page.

(In reply to Axel Hecht [:Pike] from comment #8)
> You don't care about UI locale, but content locale.

Absolutely.

I also think we'd want to avoid letting localizers write CSS.

Comment 10

[Blake Winton (:bwinton) (:☕️)]

Okay, so how do you all feel about the following approach:
If we can detect the content locale(s), and they are all in a whitelist of known-good locales, we use the custom fonts.

Alternatively, any other suggestions would be very welcome.  :)

Comment 11

[Michael Maslaney [:mmaslaney] ([e-mail])]

UPDATE:

I found "Noto", a typeface with both Serif and Sans Serif faces that has global support. 

Source: 
https://www.google.com/get/noto/

Could we utilize this instead?

Comment 12

[Michael Maslaney [:mmaslaney] ([e-mail])]

Support Information

http://blog.chromium.org/2014/10/noto-on-chromeos-no-more-tofu.html

http://googledevelopers.blogspot.com/2014/07/noto-cjk-font-that-is-complete.html

Comment 13

[Axel Hecht [:Pike]]

We use Noto fonts on Firefox OS for a few languages. https://github.com/mozilla-b2g/moztt/ in combination with all.js for b2g gives you a good idea which languages should be rendered in which font, according to our community.

Also, all those fonts in the download are 360M, didn't try how much of that is the actual font data.

Comment 14

[:Gijs (he/him)]

(In reply to Michael Maslaney [:mmaslaney] ([e-mail]) from comment #11)
> UPDATE:
> 
> I found "Noto", a typeface with both Serif and Sans Serif faces that has
> global support. 
> 
> Source: 
> https://www.google.com/get/noto/
> 
> Could we utilize this instead?

I don't know; seems like Pike just gave a good answer, though I'll say that 360M is way, way too much to be shipping with Firefox just for slightly nicer fonts. For concrete guidelines as to size, I'd recommend talking to Gavin.

Comment 15

[:Gijs (he/him)]

I think the outcome here and through discussion (in fx-dev among other places) was essentially wontfix. If I misremember or am wrong, do correct me!