Skip to content
This repository has been archived by the owner on Sep 7, 2020. It is now read-only.

Boilerplate: add favicon #207

Closed
MoOx opened this issue Feb 27, 2016 · 9 comments
Closed

Boilerplate: add favicon #207

MoOx opened this issue Feb 27, 2016 · 9 comments

Comments

@MoOx
Copy link
Owner

MoOx commented Feb 27, 2016

No description provided.

@MoOx MoOx changed the title add favicon in boilerplate and document this kind of this (react-helmet) add favicon in boilerplate and document this kind of things (react-helmet) Mar 4, 2016
@MoOx
Copy link
Owner Author

MoOx commented May 1, 2016

react-helmet has been documented in 2665fc7

@MoOx
Copy link
Owner Author

MoOx commented May 1, 2016

Do you think we should add a generic favicon?! Note sure it's a good idea, but at the same time it seems legit. But if we do so, we should add sort of apple-touch stuff? Should we?

@MoOx MoOx changed the title add favicon in boilerplate and document this kind of things (react-helmet) Boilerplate: add favicon in May 1, 2016
@MoOx MoOx changed the title Boilerplate: add favicon in Boilerplate: add favicon May 1, 2016
@thangngoc89
Copy link
Contributor

thangngoc89 commented May 2, 2016

Good point. But I think we should add phenomic's favicon. No need for a generic one. It's easier for new user to replace it :) .

we should add sort of apple-touch stuff

👍

In fact, this is a full list of apple-touch icons. https://github.com/VinSpee/vinspee.me/blob/master/web_modules/LayoutContainer/index.js

MoOx added a commit that referenced this issue May 23, 2016
@thangngoc89
Copy link
Contributor

The implementation should be straight forward. You can use this commit as your guide c7bd549

@MoOx
Copy link
Owner Author

MoOx commented Sep 15, 2016

I don't think it's "that" simple. I want to ship a best practice. So we should include favicons/ios/android/edge icons. So maybe use http://realfavicongenerator.net/ engine (I think there is siomething in node, we could even make a loader or a simple script to automate this).

@dab
Copy link

dab commented Oct 7, 2016

Started to look into this issue. Found https://github.com/jantimon/favicons-webpack-plugin
It's working fine, but slow down build process a lot. Will check for other node-based solutions. Also it can be good idea to check if icons already exists, so no need to regenerate it each time.
The other question here — how to inject links markup to our html. It can be done manually, if we will use some predefined and constant name for it, or we can use some generative approach?

@MoOx
Copy link
Owner Author

MoOx commented Oct 12, 2016

@dab feel free to take a look to https://realfavicongenerator.net/ node api (see link in the footer, upload a random image and grab the code)

@bloodyowl
Copy link
Contributor

Going to be fixed with #925

@MoOx
Copy link
Owner Author

MoOx commented May 11, 2017

v1 will have a "public" folder feature.

@MoOx MoOx closed this as completed May 11, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants