Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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

Merged
merged 2 commits into from
Dec 12, 2017
Merged

Conversation

dstansby
Copy link
Member

@dstansby dstansby commented Dec 6, 2017

No description provided.

@@ -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.'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pyto

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Huh?

Copy link
Contributor

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)

*bbox* can be

- A `BboxBase` instance
- A tuple of ``[left, bottom, width, height]`` in the given transform
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use parentheses for tuples? :)

Copy link
Member

@jklymak jklymak left a 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....


*bbox* can be

- A `BboxBase` instance
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

~.BboxBase

@@ -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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

~.DraggableLegend

@jklymak
Copy link
Member

jklymak commented Dec 12, 2017

@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...

@jklymak jklymak merged commit e68e9dc into matplotlib:master Dec 12, 2017
@QuLogic QuLogic added this to the v2.2 milestone Dec 12, 2017
@dstansby dstansby deleted the legend-docstrings branch December 12, 2017 10:12
@dstansby
Copy link
Member Author

I'm always happy for small doc fixes to be pushed to my branches, thanks a lot!

@QuLogic QuLogic modified the milestones: needs sorting, v2.2.0 Feb 12, 2018
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.

5 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.