Skip to content
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

Icon Request: icon-hacker-news-sign #1197

Closed
swanson opened this issue May 29, 2013 · 5 comments
Closed

Icon Request: icon-hacker-news-sign #1197

swanson opened this issue May 29, 2013 · 5 comments

Comments

@swanson
Copy link

swanson commented May 29, 2013

Along the same lines as the github/twitter/facebook sign icons, would be nice to have one for the Hacker News "Y"

I've rigged something together with the blank sign and a span but it seems hacky and doesn't scale.

image

@mjbaldwin
Copy link

As someone who would rather not see "icon bloat" (so I don't have to deal with subsetting the font)... I think that Hacker News is both such a "niche" site (compared to Twitter, for example), and so easy to implement without an extra icon (just slap a normal "Y" on top of icon-sign-blank -- it's not like it even has an official logo, really), that a separate icon for it is not really worth the extra bytes for all the sites that use FontAwesome. (As much as I love Hacker News, BTW!)

@swanson
Copy link
Author

swanson commented Jun 13, 2013

Devil's advocate: Trello and Gittip certainly seem more "niche" than HN.

I just figured that since the fontawesome HN user always posts new version to HN and they get to the front page of HN that an HN icon made sense :)

@mjbaldwin
Copy link

I totally agree with you about Trello and Gittip :)

Yeah... I almost wonder if, with some of the icons, at some sense it'll make sense to split them into separate files. Dunno, depends on how much it grows.

@swanson
Copy link
Author

swanson commented Jun 13, 2013

Or a new brand only - FontAwesomeLogo font :)

davegandy added a commit that referenced this issue May 14, 2014
davegandy added a commit that referenced this issue May 14, 2014
@davegandy
Copy link
Member

Added to 4.1.0-wip branch. Closing this request.

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

No branches or pull requests

3 participants