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

Ensure that fonts are present on travis when building docs. #3950

Merged
merged 5 commits into from Dec 29, 2014

Conversation

jenshnielsen
Copy link
Member

If we publish the documents build from Travis it makes sense to ensure that all fonts are available
when building docs.

This fixes the docs build by:

  • Add free fallback fonts to examples that uses Courier and Helvetica
  • Add Felipa as an alternative cursive font. None
    of the current cursive fonts are free as far as I can see.
  • Install Humor Sans and Felipa on Travis.

The results are available at http://jenshnielsen.github.io/mplddocs/examples/pylab_examples/text_handles.html, http://jenshnielsen.github.io/mplddocs/examples/pylab_examples/fonts_demo.htm, http://jenshnielsen.github.io/mplddocs/examples/showcase/xkcd.html and http://jenshnielsen.github.io/mplddocs/examples/pylab_examples/logo.html

@tacaswell
Copy link
Member

This does raise the question if the main docs are screwed up because I built them on my system which may not have all of the right fonts.

Compare http://matplotlib.org/examples/pylab_examples/fonts_demo.html and http://jenshnielsen.github.io/mplddocs/examples/pylab_examples/fonts_demo.html I am not sure where the changes in text position came from..

@tacaswell tacaswell added this to the v1.5.x milestone Dec 27, 2014
@jenshnielsen
Copy link
Member Author

@tacaswell I updated the positions and relaxed the font requirements in #3801.
Might be worth back porting to 1.4.3 ?

It looks like the only issue with the main docs is a missing Cursive font. Prior to #3801 the only working Cursive font was Script MT which was hard coded into the example.

@pelson
Copy link
Member

pelson commented Dec 28, 2014

LGTM 👍

tacaswell added a commit that referenced this pull request Dec 29, 2014
DOC/TST : Ensure that fonts are present on travis when building docs.
@tacaswell tacaswell merged commit f0bb0ab into matplotlib:master Dec 29, 2014
@jenshnielsen jenshnielsen deleted the travis_text_fixes branch December 29, 2014 15:04
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.

None yet

3 participants