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

[PR] Remove custom OpenSans fonts #309

Merged
merged 3 commits into from
Aug 31, 2015
Merged

[PR] Remove custom OpenSans fonts #309

merged 3 commits into from
Aug 31, 2015

Conversation

jeremyfelt
Copy link
Contributor

We have not relied on this in a while and have communicated our desire to stop hosting these files in favor of those hosted by Google.

We'll communicate again via web.wsu.edu when we merge this into develop.

See #162

We are no longer serving these on repo.wsu.edu after deprecating
how these were provided as of 2014
There was no actual handling in the Spine for turning OpenSans on
or off, but the styles and fonts were available to use. A long
while ago we started encouraging the use of Google's OpenSans font
files instead as they were more comprehensive and manageable.
@jeremyfelt jeremyfelt self-assigned this Aug 3, 2015
@jeremyfelt jeremyfelt added this to the Next Release milestone Aug 3, 2015
@jeremyfelt
Copy link
Contributor Author

Let's keep this unmerged for the time being just in case something else needs to ship right away.

Rather than generating a bunch of 404s for the missing opensans
file. Let's start by emptying the file of info that will actually
call fonts.
jeremyfelt added a commit that referenced this pull request Aug 31, 2015
@jeremyfelt jeremyfelt merged commit 6570b69 into develop Aug 31, 2015
@jeremyfelt jeremyfelt deleted the remove-opensans branch August 31, 2015 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant