Skip to content

Conversation

@talbs
Copy link
Contributor

@talbs talbs commented Dec 16, 2013

This work syncs up how the LMS loads Open Sans (a typeface we use throughout edX's UI) to how its loaded/referenced elsewhere (Studio, Certificates, code.edx.org) - through Google Web Fonts. Key changes include:

@talbs
Copy link
Contributor Author

talbs commented Dec 16, 2013

@frrrances and @dianakhuang, mind taking a look?

Also, this may be held up by #1961, or in some way allowing us to specify protocol-relative URLs in CSS files without Django-Pipeline falling over.

* uses Google Web Fonts reference rather than local files
* removes unused local font files
* adds in semi-bold weight support
@singingwolfboy
Copy link
Contributor

FYI: this pull request is now good to go -- we can specify protocol-relative URLs in CSS.

@frrrances
Copy link
Contributor

Lookin good, gettin presents. 👍

talbs added a commit that referenced this pull request Dec 20, 2013
LMS: Use Open Sans Typeface via Google Web Fonts
@talbs talbs merged commit 6d6bcb7 into master Dec 20, 2013
@talbs
Copy link
Contributor Author

talbs commented Dec 20, 2013

Thanks for the http://www.marriedtothesea.com/031908/lookin-good-gettin-presents.gif! Brought this to "Mergetown".

@talbs talbs deleted the talbs/lms-fonts branch December 20, 2013 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants