Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
MEP12 text alignment example #8228
+12
−10
Conversation
dstansby
added some commits
Mar 7, 2017
dstansby
added the
Documentation
label
Mar 7, 2017
NelleV
changed the title from
MEP12 text alignment example to [MRG+1] MEP12 text alignment example
Mar 7, 2017
| @@ -47,24 +49,24 @@ | ||
| verticalalignment='top', | ||
| transform=ax.transAxes) | ||
| -ax.text(left, 0.5*(bottom + top), 'right center', | ||
| +ax.text(left, 0.5 * (bottom + top), 'right center', |
thisch
Mar 7, 2017
Contributor
I'm -1 on this and the following changes. Note that PEP8 AFAIK does not recommend to add whitespace before and after this operator.
anntzer
Mar 7, 2017
Contributor
Actually it does, see https://www.python.org/dev/peps/pep-0008/#other-recommendations.
anntzer
Mar 7, 2017
Contributor
"If operators with different priorities are used, consider adding whitespace around the operators with the lowest priority(ies)."
thisch
Mar 12, 2017
Contributor
In the above line the operator with the lowest priority is the plus operator around which whitespace is added. PEP8 does not say anything about adding whitespace to all operators independent of their priorities.
| You can precisely layout text in data or axes (0,1) coordinates. This | ||
| example shows you some of the alignment and rotation specifications to | ||
| -layout text | ||
| +layout text. |
anntzer
merged commit 355695a
into matplotlib:master
Mar 8, 2017
|
Thanks! |
anntzer
changed the title from
[MRG+1] MEP12 text alignment example to MEP12 text alignment example
Mar 8, 2017
dstansby
deleted the
dstansby:alignment-example branch
Mar 8, 2017
QuLogic
added this to the
2.1 (next point release)
milestone
Apr 12, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
dstansby commentedMar 7, 2017
Another documentation update!