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

Default changes: legend #5269

Closed
Tillsten opened this issue Oct 18, 2015 · 6 comments
Closed

Default changes: legend #5269

Tillsten opened this issue Oct 18, 2015 · 6 comments

Comments

@Tillsten
Copy link
Contributor

I actually think that the current defaults for legend too spacious. Reducing the fontsize, spacing and padding is what i am thinking of. The current setting is only usable if one only has one or two items, and even then i think the legend is to big. Also the location should default to 'best'.

@jenshnielsen jenshnielsen added this to the next major release (2.0) milestone Oct 18, 2015
@mdboom
Copy link
Member

mdboom commented Oct 19, 2015

Thanks. We'll take this all into consideration as we change defaults for 2.0

@lstagner
Copy link

May I also suggest defaulting to frameon=False. I often feel the frame adds unnecessary information to the plot.

@efiring
Copy link
Member

efiring commented May 2, 2016

I'm inclined to agree with the suggestions here. I often have to reduce the legend size. Leaving the frame out by default would fit in with the idea of minimizing inessential elements.
I'm not sure whether this issue was simply dropped by accident, or whether a decision was made not to act on it--I'm guessing it was the former. But maybe not; there were some changes to legend (default location is now 'best', and the box is fancy). @Tillsten, what default values would you suggest?

@tacaswell
Copy link
Member

This just fell off the radar.

@efiring
Copy link
Member

efiring commented May 2, 2016

To elaborate, the legend.fontsize is 'large' while the axes.labelsize is 'medium'--that definitely accounts for some of the sense of imbalance!

efiring added a commit to efiring/matplotlib that referenced this issue May 15, 2016
efiring added a commit to efiring/matplotlib that referenced this issue May 16, 2016
efiring added a commit to efiring/matplotlib that referenced this issue May 17, 2016
efiring added a commit to efiring/matplotlib that referenced this issue May 23, 2016
@efiring
Copy link
Member

efiring commented May 28, 2016

Closed by #6424.

@efiring efiring closed this as completed May 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants