-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
add SVG version of new logo #884
Conversation
The gradients need some slight tuning, will follow up. |
Colors are only slightly different, I had to check with a color picker, can't see the difference with my eyes so looks good to me. |
00d5ce4
to
3b6c879
Compare
Did a few minor tweaks. I'd say it's pretty flawless now. Merge at will. |
Can someone weigh in? The gradient difference is negligible imho, LGTM. |
FYI: I just merged #880 to get the logo issue fixed while you're fine-tuning the svg version. As soon as it's ready, please go for it! |
I'll do one more pass on tuning the gradients. |
I really appreciate the effort here, but can't we ask whoever designed the logo to give us the updated |
@fhemberger gotcha, ignore my previous comment then. |
Thanks for making this, @silverwind. I was surprised to find that only AI files are available for download on the website. It would be great to also link to this new SVG file from the https://nodejs.org/en/about/resources/ page. |
cc @mikeal, who said on #526 (comment)
|
077fb68
to
78ba31e
Compare
78ba31e
to
206a51e
Compare
Did a few very minor tweaks to the main gradient before the merge. I'll go ahead and replace the old PNG in another PR. |
My attempt at converting https://github.com/nodejs/nodejs.org/blob/master/static/images/logos/nodejs-new-pantone-white.ai to a small size SVG. I suspect the colors in the source file are off a bit (they are pretty dull on the 'o'), so I went ahead and manually rewrote the gradient
<stop>
s. Note that the gradients are a approximation. Also, I cropped the viewBox.