Skip to content

Use a smaller image for the header logo - #880

Merged
fhemberger merged 2 commits into
masterfrom
gh-877
Sep 6, 2016
Merged

fhemberger merged 2 commits into
masterfrom
gh-877

Conversation

@lpinca

@lpinca lpinca commented Aug 30, 2016 •

Copy link
Copy Markdown
Member

This reduces the size of the logo used in the header.
The new size is 245 x 150 which is 2x (for retina displays) the maximum size the image can be.
The image has been compressed with ImageOptim.

@ghost

ghost commented Aug 30, 2016

Copy link
Copy Markdown

LGTM

@fhemberger

Copy link
Copy Markdown
Contributor

Compressed the image … LGTM

@lpinca

lpinca commented Aug 30, 2016

Copy link
Copy Markdown
Member Author

I think it's possible to shave some more bytes with zopflipng 😄.

@silverwind

Copy link
Copy Markdown
Contributor

LGTM. It'd be awesome if it were a SVG, but I don't suppose a vector source image is available?

@lpinca

lpinca commented Aug 31, 2016 •

Copy link
Copy Markdown
Member Author

@silverwind there is a vectorial version (nodejs-new-pantone-white.ai) which could serve the purpose.

@lpinca

lpinca commented Aug 31, 2016 •

Copy link
Copy Markdown
Member Author

I tried to convert nodejs-new-pantone-white.ai to SVG with Inkscape but the colors look different. Here is the result https://dl.dropboxusercontent.com/u/58444696/nodejs-new-pantone-white.svg.

Can someone try with Illustrator?

@silverwind

Copy link
Copy Markdown
Contributor

I too only have inkscape available, but it seems the dull colors are there in the source .ai as well. Below is an attempt at conversion, I'd wager the gradient <stop>s could be removed completely, so it's a WIP at best.

@silverwind

Copy link
Copy Markdown
Contributor

#884 for my version of the logo.

@fhemberger
fhemberger merged commit c73c74a into master Sep 6, 2016
@fhemberger
fhemberger deleted the gh-877 branch September 6, 2016 08:30
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