851513 - Replace MozTT with Feura Sans

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[Patryk Adamczyk [:patryk] UX]

The vendor will send us a tweaked version of the v.1.0 font. We will need to replace every instance of MozTT in the CSS files and reference Feura Sans instead.

Sam please work with Micheal to resolve this when the font comes in, in about a week.

Comment 1

[Lukas Blakk [:lsblakk] use ?needinfo]

Mass edit to set tracking-b2g18+ for these UX bugs that were called out for v1.1

Comment 2

[Michael Wu [:mwu]]

Any news? It's been a week past when we expected the new fonts to arrive.

Comment 3

[Michael Wu [:mwu]]

I've received Feura Sans and it's checked into moztt.

https://github.com/mozilla-b2g/moztt/commit/0cd8e64042e2ef1d4f3191806170be9bd9dd1857

Comment 4

[Sam Joch [:samjoch]]

Can we use it as 'MozTT' in css stylesheets ?

Comment 5

[Michael Wu [:mwu]]

Nope. It's actually named "Feura Sans". We'll need to switch the stylesheets over to the new name. It'll probably be a good idea to leave MozTT in the list to make the transition more graceful.

Comment 6

[Michael Wu [:mwu]]

Removed MozTT from the package list:

https://github.com/mozilla-b2g/gonk-misc/commit/40523f3a65bf8841372a3b5298a9afe069de2d86

Comment 7

[Stephany Wilkes]

What happens next with this one?

Comment 8

[Sam Joch [:samjoch]]

I am going to add FeuraSans directly in gecko and make it the default font in Gaia.

Comment 9

[Michael Wu [:mwu]]

(In reply to Sam Joch [:samjoch] from comment #8)
> I am going to add FeuraSans directly in gecko and make it the default font
> in Gaia.

The gecko side changes are ready to go - we're just making sure the fonts don't break any tests before we push them in for sure. See bug 840315 for the gecko side changes.

Comment 10

[Michael Wu [:mwu]]

Updated to Feura Sans 1.0 - https://github.com/mozilla-b2g/moztt/commit/c2cd096baa69d4f5c2473523d352fba1a94c5cf4 .

Comment 11

[Sam Joch [:samjoch]]

Michael: can you confirm you are working on getting FeuraFont the default font in Gaia ?
So i can prepare a patch where all references to 'MozTT' in gaia are deleted.

Comment 12

[Michael Wu [:mwu]]

(In reply to Sam Joch [:samjoch] from comment #11)
> Michael: can you confirm you are working on getting FeuraFont the default
> font in Gaia ?
> So i can prepare a patch where all references to 'MozTT' in gaia are deleted.

I'm only handling the gonk and gecko side of things.

Comment 13

[Sam Joch [:samjoch]]

Attachment: Gaia PR

Removing all MozTT references

Comment 14

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 737881 [details]
Gaia PR

The patch looks good. It seems like the Gecko part has been backed-out though so you will need to wait before landing this if you don't want to regress all apps (and be backed-out too :))

Comment 15

[Michael Wu [:mwu]]

It got backed out on b2g18 but stuck (after disabling annoying tests) on mozilla-central. I guess you can land it now and wait for the b2g18 side change to stick before putting it on v1-train.

Comment 16

[Michael Wu [:mwu]]

Removal of MozTT has been uplifted to v1-train https://github.com/mozilla-b2g/gonk-misc/commit/9a68fef942e8a5ce470051c6d9ff6652d2565f63 .

Sam, it should be safe to land your gaia changes, though it looks bitrotted by now.

Comment 17

[Eric Pang [:epang] UX]

(In reply to Michael Wu [:mwu] from comment #16)
> Removal of MozTT has been uplifted to v1-train
> https://github.com/mozilla-b2g/gonk-misc/commit/
> 9a68fef942e8a5ce470051c6d9ff6652d2565f63 .
> 
> Sam, it should be safe to land your gaia changes, though it looks bitrotted
> by now.

Sam, based on Michael's comment, is there anything else that needs to be done for this bug?  Usually I wait for statusb2g18 verified/fixed, but I haven't seen that yet.  Let me know if this has been completed, thanks!

Comment 18

[Sam Joch [:samjoch]]

I have problem of rendering the font with the rigth weight.
How can i be sure to have all the right required fonts in gecko ?

Comment 19

[Michael Wu [:mwu]]

Run this to list the system fonts:
adb shell ls /system/fonts

Comment 20

[Sam Joch [:samjoch]]

Landed in master: https://github.com/mozilla-b2g/gaia/commit/06490ac675db1cba40d3391a718d536164de7709

Comment 21

[Evyatar 'Tron' Amitay (everything.me)]

Guys, when making such a system-wide change, please take a second to notify us at e.me about it so we can make sure everything's alright on our end (you can just add one of us to the CC list, we'll take it from there).

opened bug 875286 to fix it.

Comment 22

[Sam Joch [:samjoch]]

Comment on attachment 737881 [details]
Gaia PR

NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Need to change font to Feura Sans.
User impact if declined: Wrong font will be used (MozTT)
Testing completed: Yes
Risk to taking this patch (and alternatives if risky): Only remove or replace MozTT reference in all gaia repo.
String or UUID changes made by this patch: -

Comment 23

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Comment on attachment 737881 [details]
Gaia PR

Let's go thought the leo? process to land on v1-train.

Comment 24

[Leo]

To leo.bugzilla.gecko 
Please check this

Comment 25

[David Scravaglieri [:scravag]]

Please let us know if there is a legal issue for using MozTT and give us a recommendation for blocking or not.

Comment 26

[Alex Keybl [:akeybl]]

We're not blocking based upon the fact that this font change is an enhancement. Let's look at a patch approval nomination once we're comfortable with the quality.

Comment 27

[Michael Wu [:mwu]]

The font replacement already happened long ago. This is mostly a cleanup AFAICT.

Comment 28

[Patryk Adamczyk [:patryk] UX]

(In reply to David Scravaglieri [:scravag] from comment #25)
> Please let us know if there is a legal issue for using MozTT and give us a
> recommendation for blocking or not.

There isn't a legal issue for using MozTT, however its outdated. Feura Sans is the font that we are distributing and contains the expanded character set.