Skip to content
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

savefig with bbox_inches='tight' takes account of all text artists (against v1.1.x) #739

Merged
merged 3 commits into from Mar 17, 2012

Conversation

leejjoon
Copy link
Contributor

@leejjoon leejjoon commented Mar 4, 2012

This is same as PR #689, but against v1.1.x branch.

@@ -8267,6 +8267,13 @@ def matshow(self, Z, **kwargs):
integer=True))
return im

def get_default_bbox_extra_artists(self):
Copy link
Member

Choose a reason for hiding this comment

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

Could do with a docstring (and on the figure method).

@pelson
Copy link
Member

pelson commented Mar 6, 2012

If this change doesn't break any tests then does that suggest there are no tests using bbox_inches='tight'? Ideally it would be nice to add one.

efiring added a commit that referenced this pull request Mar 17, 2012
savefig with bbox_inches='tight' takes account of all text artists (against v1.1.x)
@efiring efiring merged commit ce32dd2 into matplotlib:v1.1.x Mar 17, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants