/matplotlib

Add computer modern font family #6512

Merged
merged 1 commit into from Jul 10, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
default changes text
Projects
None yet
Milestone
  v2.0.0
4 participants
@srvanrell @QuLogic @tacaswell @mdboom
@srvanrell
Contributor

srvanrell commented May 31, 2016

Add Computer Modern family into rcsetup.py (requested in #5860):

  • Computer Modern Roman
  • Computer Modern Sans Serif
  • Computer Modern Typewriter

mdboom added the needs_review label May 31, 2016

srvanrell referenced this pull request May 31, 2016

Closed

Computer Modern Roman should be the default serif when using TeX backend #5860

tacaswell added this to the 2.0 (style change major release) milestone May 31, 2016

@QuLogic QuLogic added text default changes labels Jun 22, 2016

@QuLogic
Member

QuLogic commented Jun 22, 2016

It seems like no-one got around to reviewing this. I don't think there's anything controversial though as it's basically exactly what was stated in #5860.

Since the second commit is so trivial though, can you squash the two together?

Ping @zblz anyway, just in case.
@srvanrell
Contributor

srvanrell commented Jun 22, 2016

@QuLogic
Member

QuLogic commented Jun 22, 2016

Yes, just run a rebase on them.
@srvanrell
Contributor

srvanrell commented Jun 22, 2016

@srvanrell @srvanrell srvanrell Add Computer Modern font family

All checks have passed

3 successful checks
continuous-integration/appveyor/pr — AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
coverage/coveralls — Coverage increased (+0.002%) to 70.283%
Details
188ca1a
@srvanrell
Contributor

srvanrell commented Jun 22, 2016

Done
@QuLogic
Member

QuLogic commented Jun 22, 2016

OK @zblz @mdboom ?

@tacaswell tacaswell merged commit 92dbbad into matplotlib:master Jul 10, 2016
3 checks passed

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.002%) to 70.283%
Details

tacaswell removed the needs_review label Jul 10, 2016

@tacaswell tacaswell added a commit that referenced this pull request Jul 10, 2016

@tacaswell tacaswell Merge pull request #6512 from srvanrell/add-computer-modern-font-family 
Add computer modern font family
 9f9e8a3
@tacaswell
Owner

tacaswell commented Jul 10, 2016

backported to v2.x as 9f9e8a3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment