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

Skip to content

switch to Font Awesome for icons and brand logos #71

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

Merged
merged 1 commit into from
Nov 15, 2021

Conversation

jimustafa
Copy link
Contributor

This PR uses the fontawesome5 package in place of the icons and brand logos committed to the repository.

@rougier
Copy link
Member

rougier commented Aug 30, 2021

Nice, did not know font awesome was include in texlive. LGTM.

@jimustafa
Copy link
Contributor Author

Yeah, me neither. Always fun to discover a new package. A couple of the icons are slightly different than what was previously used, most notably the GitHub brand icon.

@jimustafa
Copy link
Contributor Author

jimustafa commented Oct 16, 2021

Hey @rougier, wanted to check if this PR was OK to merge. Let me know if anything needs to be addressed.

@jklymak
Copy link
Member

jklymak commented Nov 10, 2021

I forget what the merge policy is here, is it just @rougier?

@jklymak jklymak merged commit 21a2723 into matplotlib:master Nov 15, 2021
@jklymak
Copy link
Member

jklymak commented Nov 15, 2021

ooops, sorry, meant to enter a comment that these look good, and nice to get rid of a repo dependency...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants