Skip to content

Conversation

@mikeal
Copy link
Contributor

@mikeal mikeal commented Feb 18, 2016

Loving this new logo :)

@stevemao
Copy link
Contributor

@rvagg
Copy link
Member

rvagg commented Feb 19, 2016

FYI

screen shot 2016-02-19 at 1 56 13 pm

stevemao added a commit that referenced this pull request Feb 19, 2016
@stevemao stevemao merged commit 5f3872b into master Feb 19, 2016
@stevemao stevemao deleted the new-logo branch February 19, 2016 04:24
@fhemberger
Copy link
Contributor

@mikeal Please use ImageAlpha and ImageOptim to shrink the file size before adding images. You always tend to add unnecessarily large files to the site.

Updated the image, went from 63KB down to 24KB: 6b49e53.

@mikeal
Copy link
Contributor Author

mikeal commented Feb 19, 2016

ya, i always get ahead of myself and checkin giant images. sorry :/

@okuryu okuryu mentioned this pull request Feb 22, 2016
@silverwind
Copy link
Contributor

Not sure who updated the org logo on Github, but it has some artifacts which are especially noticeable on dark backgrounds:

artifacts

Also, the hexagon is very slightly cut off at the top.

@stevemao
Copy link
Contributor

It looks correct with the default theme. @silverwind are you using https://github.com/StylishThemes/GitHub-Dark? I can see there is a white area on the image though.

@silverwind
Copy link
Contributor

Yeah using that style, but the white part is visible on the left too: https://avatars1.githubusercontent.com/u/9950313. I'll see to making fixed version based on the logo in this PR.

@silverwind
Copy link
Contributor

Here's a version that's cleanly trimmed and square. Someone please update the org logo!

node-org

By the way, The gradients on the original source are not really smooth. Maybe a compression artifact or is it intended?

@mikeal
Copy link
Contributor Author

mikeal commented Feb 23, 2016

I just cleaned this up, should look better now.

@silverwind
Copy link
Contributor

Looking great, thanks 👍

f

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.

5 participants