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

Fix #4333: Whitespace after sub/super cluster by mdboom · Pull Request #4401 · matplotlib/matplotlib · GitHub

Repository navigation

Fix #4333: Whitespace after sub/super cluster - #4401

Merged
jenshnielsen merged 4 commits into
matplotlib:masterfrom
mdboom:whitespace-mathtext
May 8, 2015
Merged

jenshnielsen merged 4 commits into
matplotlib:masterfrom
mdboom:whitespace-mathtext

Conversation

mdboom commented May 4, 2015

Copy link
Copy Markdown
Member

The sub/super cluster had an extra bit of space applied after it. It's not clear why this was ever there, and removing it only seems to improve the existing regression tests.

mdboom added this to the next point release milestone May 4, 2015

Copy link
Copy Markdown
Member

Can you add a note in doc/api/api_changes ?

Copy link
Copy Markdown
Member

And I agree that this improves all of those tests.

mdboom commented May 8, 2015

Copy link
Copy Markdown
Member Author

API changes entry added.

jenshnielsen added a commit that referenced this pull request May 8, 2015
Fix #4333: Whitespace after sub/super cluster
jenshnielsen merged commit e0c5379 into matplotlib:master May 8, 2015
mdboom deleted the whitespace-mathtext branch November 9, 2015 02:35
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL