900420 - Remove legacy fonts setting

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[[e-mail]]

currently there are `sans-serif` ,  `open sans` and `MozTT` inside of gaia repository.

expect:
remove unnecessary font styles, use `sans-serif` if needed.

Comment 1

[(inactive after 6/18) Alive Kuo [:alive]]

Lemme say goodbye to MozTT because I was the one who added it to gaia ;)

Comment 2

[(inactive after 6/18) Alive Kuo [:alive]]

Attachment: https://github.com/mozilla-b2g/gaia/pull/11332

1. I file another pull request to pdf.js for removing MozTT reference.

2. The following specific font-family are remained:
* PDFJS annotation-paragraph.svg
* Keyboard Symbol
* Japanese IME

Asuth, Cristian, please give me feedback about the change to email and ev.me is appropriate or not.

Comment 3

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Comment on attachment 785596 [details]
https://github.com/mozilla-b2g/gaia/pull/11332

I think you would need to uplift a patch to pdf.js

Comment 4

[Andrew Sutherland [:asuth] (he/him)]

Comment on attachment 785596 [details]
https://github.com/mozilla-b2g/gaia/pull/11332

Thanks for checking with me!  Removing the monospace line is not appropriate for the e-mail app.  The value is intentionally chosen for consistency with Thunderbird for the display of text/plain message parts.  We should only change it in direct consultation with UX about message display.  That should be happening for v1.2.

Comment 5

[(inactive after 6/18) Alive Kuo [:alive]]

(In reply to Andrew Sutherland (:asuth) from comment #4)
> We should only
> change it in direct consultation with UX about message display.  That should
> be happening for v1.2.

O.K.!

(In reply to Tim Guan-tin Chien [:timdream] (MoCo-TPE) from comment #3)
> Comment on attachment 785596 [details]
> https://github.com/mozilla-b2g/gaia/pull/11332
> 
> I think you would need to uplift a patch to pdf.js

PDF.js change is merged already.
https://github.com/mozilla/pdf.js/pull/3540

Comment 6

[(inactive after 6/18) Alive Kuo [:alive]]

master

https://github.com/mozilla-b2g/gaia/commit/1f8fc15ebcda18f8f70c835814cb5e859297bd53