Legend marker label placement

[cimarronm]

Controls horizontal marker placement so that one specify either maker first or label first in the legend. This incorporates the feature request in issue #3285.

[mdboom]

Cool!

I wonder if it would make more sense to right align everything -- this is just an aesthetic issue, so take it with a grain of salt.

Instead of (where xxx is text and XXX is markers):

xxx XXX
xxx XXX
xxxxxxx XXX

do:

    xxx XXX
    xxx XXX
xxxxxxx XXX

Then the markers are more pleasantly aligned.

[cimarronm]

@mdboom, that definitely makes sense. I'll look into modifying to right align it when markers are on the right

[tacaswell]

I am not a fan of the in-line logic, but not enough to make more fuss than leaving a comment.

[cimarronm]

Actually, I'm not either but was being lazy. I'll rework it to remove it ternary if

[tacaswell]

Can you re-base this to erase the old test images from history?

[cimarronm]

Yeah, I'll remove the old image

[tacaswell]

@cimarronm Could you re-run boilerplate.py to update pyplot with the new kwargs?

[tacaswell]

This should also get a whats_new entry to advertise the new feature.

[cimarronm]

boilerplate.py seems to be updating quite a bit of things. Seems like it may be doing something wrong for me. I'll try to look at it in more detail tonight.

[tacaswell]

It may be the case that boilerplate has not been run in a while...

[WeatherGod]

Also remember that boilerplate.py runs off of the installed version of
matplotlib, not the source tree, IIRC.

Ben Root

On Sat, Oct 18, 2014 at 1:44 PM, Thomas A Caswell notifications@github.com
wrote:

It may be the case that boilerplate has not been run in a while...

—
Reply to this email directly or view it on GitHub
#3499 (comment)
.

[cimarronm]

@tacaswell Rebased on top of latest master and ran boilerplate.py. It doesn't look like any changes are needed to pyplot.py as legend is completely declared as a *args,**kwargs vardiac:

def legend(*args, **kwargs):
    ret = gca().legend(*args, **kwargs)
    draw_if_interactive()
    return ret