FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix #5998: Support fallback font correctly by mdboom · Pull Request #6203 · matplotlib/matplotlib · GitHub

Repository navigation

Fix #5998: Support fallback font correctly - #6203

Merged
tacaswell merged 2 commits into
matplotlib:masterfrom
mdboom:mathfonts
Apr 12, 2016
Merged

tacaswell merged 2 commits into
matplotlib:masterfrom
mdboom:mathfonts

Conversation

mdboom commented Mar 21, 2016

Copy link
Copy Markdown
Member

Cc: @zblz

mdboom added this to the 2.0 (style change major release) milestone Mar 21, 2016
mdboom changed the title Fix #6015: Support fallback font correctly Fix #5998: Support fallback font correctly Mar 21, 2016
Comment thread lib/matplotlib/mathtext.py Outdated

def __init__(self, *args, **kwargs):
# This must come first so the backend's owner is set correctly
TruetypeFonts.__init__(self, *args, **kwargs)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Is the comment above still true?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Actually, I think it turned out that this move wasn't necessary after all -- I had meant to revert it before committing. I'll do so now.

zblz commented Mar 23, 2016

Copy link
Copy Markdown
Member

Looks good to me! 👍

Do we have any idea why forcing an italics class was initially needed? Or was it just a bug?

mdboom commented Mar 23, 2016

Copy link
Copy Markdown
Member Author

In the context of STIX falling back to Bakoma, it needs to be hardcoded to it because the symbols the fallback was needed for are only in it. I think this was copied-and-pasted to the context of DejaVu falling back to STIX, where that is not the case.

QuLogic commented Mar 26, 2016

Copy link
Copy Markdown
Member

Are there no italic numbers? 05 vs 15?

I also wonder what is wrong with the baseline of the 't'.

mdboom commented Apr 11, 2016

Copy link
Copy Markdown
Member Author

@QuLogic: That's correct. There are no italic circled numbers. Not sure what you mean by "05 vs. 15"

The baseline of t is a larger issue (addressed by #5414, but it will be some time to get that done). But that's not introduced by this PR.

QuLogic commented Apr 11, 2016

Copy link
Copy Markdown
Member

@QuLogic: That's correct. There are no italic circled numbers. Not sure what you mean by "05 vs. 15"

No, I didn't mean circled, just plain italic numbers, e.g., mathfont_dejavusans_05.png vs. mathfont_dejavusans_15.png specify mathrm vs. mathit. But I see this is the case for the other fonts as well.

mdboom commented Apr 11, 2016

Copy link
Copy Markdown
Member Author

Yes -- there are no italic numbers in any of these fonts.

mdboom commented Apr 11, 2016

Copy link
Copy Markdown
Member Author

This is ready to merge.

tacaswell merged commit 0db1152 into matplotlib:master Apr 12, 2016
tacaswell added a commit that referenced this pull request Apr 12, 2016
Fix #5998: Support fallback font correctly

Copy link
Copy Markdown
Member

backported to v2.x as 819f746

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

4 participants


Back | FazBrowse Home | New Git URL