-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Clean up legend docstrings #9946
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
Conversation
lib/matplotlib/legend.py
Outdated
@@ -1107,16 +1117,16 @@ def set_title(self, title, prop=None): | ||
self.stale = True | ||
|
||
def get_title(self): | ||
'return Text instance for the legend title' | ||
'Return `~.text,.Text` instance for the legend title.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pyto
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huh?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo :-)
(sorry that joke is getting old)
lib/matplotlib/legend.py
Outdated
*bbox* can be | ||
|
||
- A `BboxBase` instance | ||
- A tuple of ``[left, bottom, width, height]`` in the given transform |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use parentheses for tuples? :)
a32bc7c
to
aac4d2b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple more links to take or leave....
lib/matplotlib/legend.py
Outdated
|
||
*bbox* can be | ||
|
||
- A `BboxBase` instance |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
~.BboxBase
lib/matplotlib/legend.py
Outdated
@@ -1258,7 +1272,7 @@ def draggable(self, state=None, use_blit=False, update="loc"): | ||
* False : turn draggable off | ||
|
||
If draggable is on, you can drag the legend on the canvas with | ||
the mouse. The DraggableLegend helper instance is returned if | ||
the mouse. The `DraggableLegend` helper instance is returned if |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
~.DraggableLegend
@dstansby I just pushed to your branch (which I'm 99.9% sure you said was OK in the past). Feel free to revert if you don't like it. I'll commit if the checks pass... |
I'm always happy for small doc fixes to be pushed to my branches, thanks a lot! |
No description provided.