Created app has wrong favicon "sizes" value in the manifest file, which results in console error.

[mohsinulhaq]

Is this a bug report?

Yes

Can you also reproduce the problem with npm 4.x?

Did not try. Doesn't seem to be related to that.

Which terms did you search for in User Guide?

I didn't.

Environment

1. node -v: v8.7.0
2. npm -v: 5.5.1
3. yarn --version (if you use Yarn):
4. npm ls react-scripts (if you haven’t ejected):

Then, specify:

1. Operating system: macOS 10.13
2. Browser and version (if relevant): Chrome 61

Steps to Reproduce

1. Go to the created app's manifest file in public folder.
2. The icons array has wrongly mentioned the favicon.ico size as 192x192 instead of 16x16 size of the included favicon file.
3. Run the app. Go to Dev Tools' Application section. Open Manifest.
4. Click Add to homescreen. This results in console error:
   Error while trying to use the following icon from the Manifest: http://localhost:3000/favicon.ico (Resource size is not correct - typo in the Manifest?) Site cannot be installed: icon downloaded from the manifest was empty or corrupted

Expected Behavior

Should have searched for 144x144 image, where it would fail if we fix the size now.

Actual Behavior

Console error.

Reproducible Demo

[ryansully]

The included favicon.ico file I found was 64x64 instead of 16x16, so I used that value in #3287.

[mohsinulhaq]

@ryansully yes, you are right, it's 64x64. My IDE wrongly showed it to be 16x16. My bad.

[mohsinulhaq]

@ryansully have you signed the CLA here https://code.facebook.com/cla to get your PR approved?

[ryansully]

That's odd, I've signed it and contributed before, should I sign again?

[ryansully]

Actually, #3287 has the CLA Signed label:
#3287 (comment)

PR should be ready to go.

[camux]

I see this warning but icon size is 64x64 how remove this warning?

[abhishekakade]

I see this warning but icon size is 64x64 how remove this warning?

@camux, in your app, go to public folder, right click on favicon.ico file and select Properties, then Details tab and check Dimensions value. Then open manifest.json file (which is in the same public folder) in a code editor and under "icons", change "sizes" value to whatever dimensions the favicon.ico file had. That should fix the console error.