Repository navigation
MEP12 text alignment example - #8228
Conversation
NelleV
changed the title
| transform=ax.transAxes) | ||
|
|
||
| ax.text(left, 0.5*(bottom + top), 'right center', | ||
| ax.text(left, 0.5 * (bottom + top), 'right center', |
There was a problem hiding this comment.
I'm -1 on this and the following changes. Note that PEP8 AFAIK does not recommend to add whitespace before and after this operator.
There was a problem hiding this comment.
Actually it does, see https://www.python.org/dev/peps/pep-0008/#other-recommendations.
There was a problem hiding this comment.
It says 'Use your own judgment;'
There was a problem hiding this comment.
"If operators with different priorities are used, consider adding whitespace around the operators with the lowest priority(ies)."
There was a problem hiding this comment.
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.
anntzer
left a comment
anntzer
left a comment
There was a problem hiding this comment.
feel free to dismiss the review if "to" is more colloquial
| 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. |
There was a problem hiding this comment.
"for layout text"? (nonnative speaker here)
|
Thanks! |
anntzer
changed the title
Another documentation update!