Noissue - Update font-family to Source Sans Pro on abp.org

includes/styles.tmpl

Index: includes/styles.tmpl
===================================================================
--- a/includes/styles.tmpl
+++ b/includes/styles.tmpl
@@ -1,2 +1,3 @@
 <link rel="stylesheet" href="/css/defaults.css">
+<link rel="stylesheet" href="/css/fonts.css">

[juliandoucette, 2018/03/14 17:08:38]

Why a separate file?

[saroyanm, 2018/03/14 17:56:30]

Ideally this will go to the websites default to import it for multilingual
websites.

Note: We use this file in UI separately in order to load in multiple pages, but
I don't think you have this need in current repo.
Not necessarily this needs to be a separate file here.

[ire, 2018/03/14 18:10:31]

I did it this way because it just seemed cleaner to separate it. Same the
defaults css is separate (I think, anyway)?

Also, when we implement the build step here everything will be in one file. 

[juliandoucette, 2018/03/14 18:18:00]

Yes and no. I separated default.css so that I could update default.css when
website-defaults updates without seeming like I'd changed main.css.
Agreed.

---

I'd prefer to update website-defaults and then update defaults.css in
web.adblockplus.org in order to include this CSS in defaults.css. But that's a
super-NIT preference. It only matters in the rare case that a user who has a bad
connection is having a hard time downloading this small file (which will rarely
if ever happen).

[ire, 2018/03/15 08:30:48]

On 2018/03/14 18:18:00, juliandoucette wrote: 
I agree, but adding it to website-defaults may not be so straightforward since
we will probably have to make the path to the font files configurable. 

I will push this as it is for now and open a separate review for that

 <link rel="stylesheet" href="/css/main.css">