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

Update homepage image #2024

Merged
merged 1 commit into from May 29, 2013
Merged

Update homepage image #2024

merged 1 commit into from May 29, 2013

Conversation

mdboom
Copy link
Member

@mdboom mdboom commented May 28, 2013

The image at http://matplotlib.org/users/screenshots.html clips text and suffers from a snapping issue. I suspect both of these are fixed in master.

It'd be good to update that image (perhaps just re-run the code - or alternatively add a couple more plots...) before the v1.3.x release...

@pelson
Copy link
Member Author

pelson commented May 20, 2013

@tonysyu - I know you have some artistic flair. Any ideas? It'd be pretty cool to have a streamplot in there...

@tonysyu tonysyu mentioned this pull request May 21, 2013
@tonysyu
Copy link
Contributor

tonysyu commented May 21, 2013

Dammit Phil. Stop asking me to do things! I'm really bad at saying "no" ;)

Seriously, though: Thanks for your vote of confidence. That said, there's no way I'm touching the mathtext demo or the financial demo; they need quite a bit of cleanup.

Most of the aesthetic cleanups are pretty subtle, but I did fix some links and reword a bit of the text. For some of the plots, I did some cleanup related to MEP 12, but I wanted to avoid the hassles of moving (and re-linking) the demo files, for now.

BTW, I wanted to add a bit more spacing to the table example, but that isn't readily exposed. Also, I got confused a couple of times because a couple of examples were copied between the pylab_examples directory and the api directory. I guess this will be a part of the MEP 12 clean up.

@pelson
Copy link
Member Author

pelson commented May 21, 2013

Dammit Phil. Stop asking me to do things! I'm really bad at saying "no" ;)

I feel really bad now - I actually linked to the wrong place (I meant http://matplotlib.org/_static/logo_sidebar_horiz.png) - what a clanger!

Your change looks good though. I'm going to get it checked out and built - then I think we're pretty close to getting it merged...

Awesome work as ever @tonysyu.

@tonysyu
Copy link
Contributor

tonysyu commented May 21, 2013

Well now the issue title makes a lot more sense!

Ha! I think I indirectly chose the plots in the sidebar---except for that polar plot. They were originally part of a logo submission I made 5 years ago! I'm not sure I'll have time to fix this up before the release, but I'll look into it soon.

BTW, if the PR is too big of a change right before release, it's not a big deal. Cheers!

@ghost ghost assigned mdboom May 24, 2013
@mdboom
Copy link
Member

mdboom commented May 24, 2013

Self-assigning. Hopefully I'll get to this today.

mdboom added a commit that referenced this pull request May 29, 2013
@mdboom mdboom merged commit 493fa46 into matplotlib:v1.3.x May 29, 2013
@mdboom mdboom deleted the new_logo branch August 7, 2014 13:52
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.

None yet

3 participants