MEP12 text alignment example #8228
Conversation
| @@ -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', | |||
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.
| 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! |
Another documentation update!