Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix API links #54

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Mar 8, 2021
Merged

Fix API links #54

merged 6 commits into from
Mar 8, 2021

Conversation

drammock
Copy link
Contributor

@drammock drammock commented Feb 27, 2021 •
edited
Loading

I noticed a couple of the API links were wrong, so I've corrected them here. In the process I also:

  • cleaned some trailing whitespaces
  • standardized the fontspec defs in cheatsheet.tex

WIP because I'm still apparently failing to load all fonts correctly; the text in the "legend placement" section is messed up and is causing the whole thing to spill onto a third page.

jklymak
jklymak reviewed Feb 27, 2021
View reviewed changes
@drammock
Copy link
Contributor Author

drammock commented Feb 27, 2021 •
edited
Loading

OK I updated the URLs to use .../stable/.... What I thought was a font problem at first glance was in fact the result of bb2d3d8 not including a recompile of the PDF... it caused some lines to be too long, which bumped a section to the next column... etc. I fixed that by cherry-picking the commit from #40, which can now be closed if this gets merged rebasing against master now that #40 has been merged.

@drammock drammock changed the title Fix API links Fix API links; fix legend placement text; add simple Makefile Feb 27, 2021
@drammock drammock marked this pull request as ready for review February 27, 2021 14:08
@drammock drammock changed the title Fix API links; fix legend placement text; add simple Makefile Fix API links Mar 2, 2021
@rougier rougier merged commit afebd6b into matplotlib:master Mar 8, 2021
@rougier
Copy link
Member

rougier commented Mar 8, 2021

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jklymak jklymak jklymak left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@drammock @rougier @jklymak