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

Change to install Roboto fonts by npm #986

Merged
merged 1 commit into from
Aug 12, 2017

Conversation

roramirez
Copy link
Contributor

Remove the Roboto fonts into source code or project. These are install by npm.

@roramirez roramirez force-pushed the roboto-fonts-as-vendor branch from f7a845e to aed09f0 Compare August 8, 2017 00:45
@MichMich MichMich merged commit 2294d72 into MagicMirrorOrg:develop Aug 12, 2017
@roramirez roramirez deleted the roboto-fonts-as-vendor branch August 13, 2017 07:08
@QNimbus
Copy link
Contributor

QNimbus commented Aug 20, 2017

If I do a fresh install of MagicMirror (and npm install), the Roboto fonts will not load because the filenames in the roboto.css file do not match the actual font file names.

Example from roboto.css :
url("node_modules/roboto-fontface/fonts/roboto-condensed/RobotoCondensed-Light.ttf") format("truetype");

Actual filename is :
/node_modules/roboto-fontface/fonts/roboto-condensed/Roboto-Condensed-Light.ttf

I just wanted to verify that others can reproduce this issue as well before I proposed a fix for this...

@roramirez Can you confirm that your fonts do not get loaded as well?

@roramirez
Copy link
Contributor Author

@QNimbus You're right! My mistake :(

There a way to make a test for the css?

QNimbus added a commit to QNimbus/MagicMirror that referenced this pull request Aug 21, 2017
Fixed some font file paths
QNimbus added a commit to QNimbus/MagicMirror that referenced this pull request Aug 21, 2017
Fixed some font file paths
@QNimbus
Copy link
Contributor

QNimbus commented Aug 21, 2017

@roramirez I've made an attempt at an e2e test case and added it to the PR #986. I think I inadvertently also found the reason why the vendor_spec.js test case isn't working at this moment (hit and miss) and I will try to test my theory; if it works I will submit a PR for that as well...

@roramirez
Copy link
Contributor Author

ryck added a commit to ryck/MagicMirror that referenced this pull request Sep 17, 2017
* upstream/develop: (71 commits)
  Update sample address
  set version of express-ipfilter:
  Describe default in comment of sample config
  More secure defaults.
  Fix grunt errors
  Weather forecast settings match current weather
  Fix JavaScript error in weatherforecast
  Update fr.json
  TestSuite: Show the missing strings for translations files.
  Update fr.json with added translations
  Added e2e test for font files
  Fix issue MagicMirrorOrg#986
  Fix extra tab file translations/zh_cn.json
  Add Changelog entry for new e2e test for helloworld module
  Small change for re-running Travis CI
  Change Two Conditionals to UpperCase-If
  Fix linting?
  Show Scale of Temperature if config flag is set
  Change to install Roboto fonts by npm
  Update stylelint package
  ...
ryck added a commit to ryck/MagicMirror that referenced this pull request Sep 17, 2017
* develop: (71 commits)
  Update sample address
  set version of express-ipfilter:
  Describe default in comment of sample config
  More secure defaults.
  Fix grunt errors
  Weather forecast settings match current weather
  Fix JavaScript error in weatherforecast
  Update fr.json
  TestSuite: Show the missing strings for translations files.
  Update fr.json with added translations
  Added e2e test for font files
  Fix issue MagicMirrorOrg#986
  Fix extra tab file translations/zh_cn.json
  Add Changelog entry for new e2e test for helloworld module
  Small change for re-running Travis CI
  Change Two Conditionals to UpperCase-If
  Fix linting?
  Show Scale of Temperature if config flag is set
  Change to install Roboto fonts by npm
  Update stylelint package
  ...

# Conflicts:
#	CHANGELOG.md
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