8000 Doc build fixes by mdboom · Pull Request #5288 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]

Skip to content

Doc build fixes #5288

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

Merged
merged 5 commits into from
Oct 22, 2015
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
< 10000 div class="select-menu-loading-overlay" aria-busy="true"> Loading
Diff view
Diff view
Next Next commit
Don't use cursive and fantasy in font demo
It just produces a meaningless warning on most systems
  • Loading branch information
mdboom committed Oct 20, 2015
commit c30b74ec47f8bcec8d9d4f9d70bf269dd9eee309
2 changes: 1 addition & 1 deletion examples/pylab_examples/fonts_demo.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
alignment = {'horizontalalignment': 'center', 'verticalalignment': 'baseline'}
# Show family options

families = ['serif', 'sans-serif', 'cursive', 'fantasy', 'monospace']
families = ['serif', 'sans-serif', 'monospace']
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any particular reason to drop these. While they may not be the most beautiful fonts Travis does include them correctly here http://matplotlib.org/devdocs/examples/pylab_examples/fonts_demo.html

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah. I removed them just because they give a doc build warning on my machine. But if it works on Travis, might as well reinstate. I'll just revert this change.

Copy link
Member
< 6558 /div>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I spent some time at some point finding fonts for all the categories that are free and installed on Travis. See the .travis file. Before that it was a bit of a mess


font1 = font0.copy()
font1.set_size('large')
Expand Down
0