-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Change legend
to accept alpha
instead of (only) framealpha
.
#6363
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
Comments
The problem is that there is much more in a legend than the frame; a plain "alpha" kwarg would imply changing the alpha of all components of the legend. |
I am with @efiring on this on. |
Now, that being said, is having a "alpha" keyword to adjust the alpha of On Tue, May 3, 2016 at 7:04 PM, Thomas A Caswell notifications@github.com
|
Yeah, I totally see your points. At the same time, I feel like the effect of |
This is a common sort of trade off in API design, the 'do what I mean' vs 'do what I told you'. The upside of mapping That said, looking at the |
I'm going to close this. For the time being, I think we need to stick with the moderately ugly but descriptive |
It seems this parameter name comes from the internal parameter being set,
e.g. leg.get_frame().set_alpha(0.5)
, but it seems like it would make much more sense from an API perspective if one could simply use the argumentalpha
, i.e.plt.legend(alpha=0.5)
instead ofplt.legend(framealpha=0.5)
.The text was updated successfully, but these errors were encountered: