-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Mathtext tutorial fixes #12149
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
Mathtext tutorial fixes #12149
Conversation
fc588cc
to
b15a24e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
I've added some oxford commas and capitalization of Matplotlib. Will merge once the CI passes!
Sorry, something went wrong.
All reactions
yay, to, oxford, commas, |
All reactions
Sorry, something went wrong.
tutorials/text/mathtext.py
Outdated
|
||
Fractions, binomials and stacked numbers can be created with the ``\frac{}{}``, | ||
Fractions, binomials, and stacked numbers can be created with the ``\frac{}{}``, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line too long with the comma 😄
Sorry, something went wrong.
All reactions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm now wondering if I should use the github interface or give up on being lazy and clone the code…
Sorry, something went wrong.
All reactions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
decision postponed after the coffee break ☕
Sorry, something went wrong.
All reactions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
Sorry, something went wrong.
All reactions
- Remove reference to (obsolete) narrow python builds. - Quote a string.
- Switch from part-string, part-comment to a single large string. - Reflow to 79 characters wide.
…149-on-v3.0.x Backport PR #12149 on branch v3.0.x (Mathtext tutorial fixes)
…149-on-v3.0.0-doc Backport PR #12149 on branch v3.0.0-doc (Mathtext tutorial fixes)
Mathtext tutorial fixes Conflicts: .flake8 - does not exist on this branch tutorials/text/mathtext.py - keep all backported changes
Successfully merging this pull request may close these issues.
None yet
PR Summary
I'd suggest reviewing the two commits separately :p
Preliminary work to a proper fix of #12108 / #12128
PR Checklist