834244 - Update Fonts on B2G

Status: RESOLVED FIXED

(Firefox OS Graveyard :: General, defect)

Description

[Patryk Adamczyk [:patryk] UX]

Hello, can we please remove:
- AndroidClock_Highlight.ttf   
- AndroidClock_Solid.ttf         
- AndroidClock.ttf               
- Clockopia.ttf 
- DroidSans-Bold.ttf
- DroidSans.ttf
- DroidSansMono.ttf
- DroidSerif-BoldItalic.ttf      
- DroidSerif-Bold.ttf            
- DroidSerif-Italic.ttf          
- DroidSerif-Regular.ttf
- Roboto-Bold.ttf
- Roboto-BoldItalic.ttf
- Roboto-Regular.ttf
- Roboto-Italic.ttf

Attached are higher quality / updated open source fonts.
1. Charis - Serif
2. Adobe Source Code - Monospaced
3. Roboto - Updated with all weights
4. Adobe Source Sans

Comment 1

[Patryk Adamczyk [:patryk] UX]

Attachment: Charis

Comment 2

[Patryk Adamczyk [:patryk] UX]

Attachment: Roboto (Updated)

Comment 3

[Patryk Adamczyk [:patryk] UX]

Attachment: Source Code Pro

Comment 4

[Patryk Adamczyk [:patryk] UX]

Attachment: Source Sans Pro

Comment 5

[Michael Wu [:mwu]]

A quick note - adding new fonts should be easy but removing them may be more difficult. The default android fonts are a bit more deeply embedded into the Android build system and we also make assumptions about Droid Sans being around. IIRC, Droid Sans is also the default Sans font so we would have to pick another default font if we want to eliminate Droid Sans.

I don't know how difficult updating roboto will be - I'll check.

Comment 6

[Patryk Adamczyk [:patryk] UX]

We may want to do this in a few rounds (bugs)
Lets remove the Droid Serif, and specify Charis as the default serif.
Roboto can just be copied over with the newer versions.
Adobe Source Code Pro should be the default monospaced.

And we're working with the type designer to expand the character set of MozTT (which will likely be renamed to Feura Sans), and that will be our default typeface for everything. I am hoping we can get this done v.1.1.

Comment 7

[Michael Wu [:mwu]]

Ok I have a plan for this. This bug will be for putting the fonts in the build and there will be another bug for changing the font defaults.

Patryk, can you link where you got these fonts from? Just the fonts isn't enough - we need licensing info and it would be easiest if you pointed out where these fonts came from and what version you want to use.

Comment 8

[Patryk Adamczyk [:patryk] UX]

Adobe Source Fonts.

http://store1.adobe.com/cfusion/store/html/index.cfm?store=OLS-US&event=displayFontPackage&code=1959

https://github.com/adobe/Source-Code-Pro/downloads

Charis
http://scripts.sil.org/cms/scripts/page.php?site_id=nrsi&id=CharisSIL_download

Roboto
http://developer.android.com/design/style/typography.html

Comment 9

[Michael Wu [:mwu]]

https://github.com/mozilla-b2g/moztt/commit/62f94a26d34c1f1e1846efd58d34363c051e8c66

This commit adds the fonts, but does not install them or remove any fonts. A commit to platform_build will be necessary to do that.

For reference, this is what the /system/fonts directory looks like after this is enabled:

root@android:/system/fonts # ls
CharisSIL-B.ttf
CharisSIL-BI.ttf
CharisSIL-I.ttf
CharisSIL-R.ttf
DroidNaskh-Regular.ttf
DroidSans-Bold.ttf
DroidSans.ttf
DroidSansArmenian.ttf
DroidSansEthiopic-Regular.ttf
DroidSansFallback.ttf
DroidSansGeorgian.ttf
DroidSansHebrew-Bold.ttf
DroidSansHebrew-Regular.ttf
DroidSansThai.ttf
MozTT-Bold.ttf
MozTT-Light.ttf
MozTT-Medium.ttf
MozTT-Regular.ttf
Roboto-Black.ttf
Roboto-BlackItalic.ttf
Roboto-Bold.ttf
Roboto-BoldCondensed.ttf
Roboto-BoldCondensedItalic.ttf
Roboto-BoldItalic.ttf
Roboto-Condensed.ttf
Roboto-CondensedItalic.ttf
Roboto-Italic.ttf
Roboto-Light.ttf
Roboto-LightItalic.ttf
Roboto-Medium.ttf
Roboto-MediumItalic.ttf
Roboto-Regular.ttf
Roboto-Thin.ttf
Roboto-ThinItalic.ttf
SourceCodePro-Black.ttf
SourceCodePro-Bold.ttf
SourceCodePro-ExtraLight.ttf
SourceCodePro-Light.ttf
SourceCodePro-Medium.ttf
SourceCodePro-Regular.ttf
SourceCodePro-Semibold.ttf
SourceSansPro-Black.ttf
SourceSansPro-BlackIt.ttf
SourceSansPro-Bold.ttf
SourceSansPro-BoldIt.ttf
SourceSansPro-ExtraLight.ttf
SourceSansPro-ExtraLightIt.ttf
SourceSansPro-It.ttf
SourceSansPro-Light.ttf
SourceSansPro-LightIt.ttf
SourceSansPro-Regular.ttf
SourceSansPro-Semibold.ttf
SourceSansPro-SemiboldIt.ttf

Note that DroidSans.ttf and DroidSans-Bold.ttf are just symlinks to Roboto.

Comment 10

[Michael Wu [:mwu]]

Pull request to flip the switch at https://github.com/mozilla-b2g/platform_build/pull/18

I also requested leo? so we can land this for 1.1.

Comment 11

[Dietrich Ayala (:dietrich)]

does this fix bug 825091?

Comment 12

[Michael Wu [:mwu]]

Apparently this wasn't in v1-train, so I uplifted it to v1-train on moztt:

https://github.com/mozilla-b2g/moztt/commit/2c7ca7d1095a68d36de504df382a5ed4104bc84d

Comment 13

[Michael Wu [:mwu]]

The platform_build changes have also landed. Closing, but we'll need snapshot updates (bug 849384).

Comment 14

[Michael Wu [:mwu]]

Had to land a followup to switch to charis sil compact.

Comment 15

[Ryan VanderMeulen [:RyanVM]]

Can we set status-b2g18:fixed on this, Michael?

Comment 16

[John Ford [:jhford] CET/CEST Berlin Time]

(In reply to Ryan VanderMeulen [:RyanVM] from comment #15)
> Can we set status-b2g18:fixed on this, Michael?

While everything is landed, the snapshots that we use haven't been deployed to our infrastructure.  I'm not sure how we should be marking this bug.

Comment 17

[Ryan VanderMeulen [:RyanVM]]

Mark it as depending on whatever snapshot update bug gets filed and set the status flags when it lands?

Comment 18

[Ryan VanderMeulen [:RyanVM]]

I believe that this is fixed on b2g18 now. Can someone confirm?
https://hg.mozilla.org/releases/mozilla-b2g18/rev/a827df06cffb

Comment 19

[viral [:viralwang]]

Hi Michael,

I found this following commit already phase in master branch, shall it also needed for v1-train?
https://github.com/mozilla-b2g/platform_build/commit/c647d5f967a5f17aaa27c792ea0ee35a2d44b113

Comment 20

[viral [:viralwang]]

Hi Michael,

in v1-train branch, we found Chinese character can not shown in some none-qcom devices.
I think it comes from the missing of /system/fonts/DroidSansFallback.ttf, shall we also need to phase in following commit for v1-train branch?
https://github.com/mozilla-b2g/platform_build/commit/c647d5f967a5f17aaa27c792ea0ee35a2d44b113

Comment 21

[Michael Wu [:mwu]]

We're going to try to update the fonts again. We don't need to uplift that backout to v1-train since we want the new font set.

Comment 22

[Michael Wu [:mwu]]

https://github.com/mozilla-b2g/moztt/commit/3769dfb6566897c8dca0abdc596728602d5fabae puts Droid Sans Fallback back in. This was accidentally removed. Thanks for pointing that out.

Comment 23

[John Ford [:jhford] CET/CEST Berlin Time]

(In reply to Michael Wu [:mwu] from comment #22)
> https://github.com/mozilla-b2g/moztt/commit/
> 3769dfb6566897c8dca0abdc596728602d5fabae puts Droid Sans Fallback back in.
> This was accidentally removed. Thanks for pointing that out.

Should we uplift this to v1-train?

Comment 24

[Michael Wu [:mwu]]

There's a number of things that have to be uplifted involving fonts. I want to uplift all the font bits once we have them all landed and verified working.

Comment 25

[Michael Wu [:mwu]]

Relanded -

https://github.com/mozilla-b2g/platform_build/commit/eef073ce712b741f799192fc86a4e009834760ab

Comment 27

[Michael Wu [:mwu]]

https://github.com/mozilla-b2g/platform_build/commit/7e18fb88e4579d8d1784f434a07ffe223f4b12db

All necessary pieces for this bug should be uplifted now. There's a lot of different pieces though - hopefully didn't miss anything.