Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TinyMCE: Updating Google Font Syntax #2019

Merged
merged 1 commit into from
Dec 21, 2013
Merged

TinyMCE: Updating Google Font Syntax #2019

merged 1 commit into from
Dec 21, 2013

Conversation

talbs
Copy link
Contributor

@talbs talbs commented Dec 20, 2013

With our upgrade to Sass a bit ago, we can now use plain CSS imports (Google's preferred syntax for embedding their web fonts) This work, building on the solid work of #1975, updates our font file import syntax in common/static/css/tiny-mce.css to :

  • match syntax used when importing Open Sans in LMS/Studio
  • simplify and move syntax to modern Google Web Font standards
  • maintain the http/https agnostic work

* matches syntax used when importing Open Sans in LMS/Studio
* simplifies and moves syntax to modern Google Web Font standards
* maintains the http/https agnostic work
@talbs
Copy link
Contributor Author

talbs commented Dec 20, 2013

@frrrances, @singingwolfboy, and @tusbar, mind taking a look?

@singingwolfboy
Copy link
Contributor

👍 seems reasonable to me.

@tusbar
Copy link
Contributor

tusbar commented Dec 20, 2013

👍 thank you!

talbs added a commit that referenced this pull request Dec 21, 2013
TinyMCE: Updating Google Font Syntax
@talbs talbs merged commit 2d8a80b into master Dec 21, 2013
@talbs talbs deleted the talbs/tinymce-fonts branch December 21, 2013 15:50
shimulch pushed a commit to open-craft/edx-platform that referenced this pull request Jan 26, 2021
* bump xblock adventure version

* bump xblock-utils
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.

3 participants