Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Convert logo from font svg into path svg#447

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
jim22k merged 3 commits intopython-graphblas:mainfromjim22k:logo_svg
Apr 26, 2023

Conversation

@jim22k
Copy link
Member

Same thing we did forgraphblas-algorithms -- change from a small svg which uses built-fonts and instead use an explicit path to remove the font challenges. This has been run thru a minifier.

@coveralls
Copy link

coveralls commentedApr 24, 2023•
edited
Loading

Coverage Status

Coverage: 99.698% (+5.6%) from 94.056% when pulling13586a8 on jim22k:logo_svg into1ed40ef on python-graphblas:main.

@eriknw
Copy link
Member

Yay, thanks@jim22k!

Which minifier did you use? One of the tools I tried when doingpython-graphblas/graphblas-algorithms#60 actually lowered the quality of the svg, but I forget which one.

https://vecta.io/nano appears to do a good job and is high quality.

@eriknw
Copy link
Member

I just minified all svgs withhttps://vecta.io/nano . I can't tell a difference. The size of the logos was reduced by another third or so.

jim22k reacted with thumbs up emoji

@eriknweriknw self-requested a reviewApril 26, 2023 14:29
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@eriknweriknweriknw approved these changes

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

@jim22k@coveralls@eriknw
[8]ページ先頭
©2009-2025 Movatter.jp