Skip to content

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

Merged
merged 4 commits into from
Oct 4, 2018
Merged

Mathtext tutorial fixes #12149

merged 4 commits into from
Oct 4, 2018

Conversation

anntzer
Copy link
Contributor

@anntzer anntzer commented Sep 18, 2018

PR Summary

  • First commit removes a reference to (obsolete) narrow python builds and adds quotes to an example string.
  • Second commit switches the format of the tutorial from half-docstring, half-comments to a single large docstring (there isn't really a reason to split it and I find the different highlighting a bit jarring) and reflows it to 79 characters wide.

I'd suggest reviewing the two commits separately :p

Preliminary work to a proper fix of #12108 / #12128

PR Checklist

  • Has Pytest style unit tests
  • Code is Flake 8 compliant
  • New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

@anntzer anntzer force-pushed the mathtext-tut branch 2 times, most recently from fc588cc to b15a24e Compare September 18, 2018 10:09
Copy link
Member

@NelleV NelleV left a 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!

@anntzer
Copy link
Contributor Author

anntzer commented Sep 18, 2018

yay, to, oxford, commas,


Fractions, binomials and stacked numbers can be created with the ``\frac{}{}``,
Fractions, binomials, and stacked numbers can be created with the ``\frac{}{}``,
Copy link
Member

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 😄

Copy link
Member

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…

Copy link
Member

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 ☕

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@tacaswell tacaswell added this to the v3.0.0-doc milestone Sep 19, 2018
anntzer and others added 4 commits October 4, 2018 15:31
- 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.
@jklymak jklymak merged commit 2c1a064 into matplotlib:master Oct 4, 2018
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Oct 4, 2018
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Oct 4, 2018
@anntzer anntzer deleted the mathtext-tut branch October 4, 2018 20:31
jklymak added a commit that referenced this pull request Oct 4, 2018
…149-on-v3.0.x

Backport PR #12149 on branch v3.0.x (Mathtext tutorial fixes)
jklymak added a commit that referenced this pull request Oct 5, 2018
…149-on-v3.0.0-doc

Backport PR #12149 on branch v3.0.0-doc (Mathtext tutorial fixes)
tacaswell pushed a commit to tacaswell/matplotlib that referenced this pull request Feb 12, 2019
Mathtext tutorial fixes

Conflicts:
	.flake8
          - does not exist on this branch
	tutorials/text/mathtext.py
          - keep all backported changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants