{"action":"created","issue":{"url":"https://api.github.com/repos/aspnet/AspNetCore/issues/13491","repository_url":"https://api.github.com/repos/aspnet/AspNetCore","labels_url":"https://api.github.com/repos/aspnet/AspNetCore/issues/13491/labels{/name}","comments_url":"https://api.github.com/repos/aspnet/AspNetCore/issues/13491/comments","events_url":"https://api.github.com/repos/aspnet/AspNetCore/issues/13491/events","html_url":"https://github.com/aspnet/AspNetCore/issues/13491","id":486047222,"node_id":"MDU6SXNzdWU0ODYwNDcyMjI=","number":13491,"title":"The favicon.ico in the templates is needlessly big","user":{"login":"DamianEdwards","id":249088,"node_id":"MDQ6VXNlcjI0OTA4OA==","avatar_url":"https://avatars1.githubusercontent.com/u/249088?v=4","gravatar_id":"","url":"https://api.github.com/users/DamianEdwards","html_url":"https://github.com/DamianEdwards","followers_url":"https://api.github.com/users/DamianEdwards/followers","following_url":"https://api.github.com/users/DamianEdwards/following{/other_user}","gists_url":"https://api.github.com/users/DamianEdwards/gists{/gist_id}","starred_url":"https://api.github.com/users/DamianEdwards/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DamianEdwards/subscriptions","organizations_url":"https://api.github.com/users/DamianEdwards/orgs","repos_url":"https://api.github.com/users/DamianEdwards/repos","events_url":"https://api.github.com/users/DamianEdwards/events{/privacy}","received_events_url":"https://api.github.com/users/DamianEdwards/received_events","type":"User","site_admin":false},"labels":[{"id":1118585221,"node_id":"MDU6TGFiZWwxMTE4NTg1MjIx","url":"https://api.github.com/repos/aspnet/AspNetCore/labels/area-mvc","name":"area-mvc","color":"5319e7","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2019-08-27T21:38:20Z","updated_at":"2019-08-27T22:49:09Z","closed_at":null,"author_association":"MEMBER","body":"The favicon.ico file currently used in all of our project templates is pretty big by favicon standards at 31.2 KB. In fact, yslow [recommends a size less than 1 KB](https://developer.yahoo.com/performance/rules.html?guccounter=1&guce_referrer=aHR0cDovL3lzbG93Lm9yZy91c2VyLWd1aWRlLw&guce_referrer_sig=AQAAADCl3oqR1ntg0swhkGy8r-1Ycllb2orjUwmDBV5sETjPf7-prhQ-NCL-3edCBDao7_gcnp9NL_RIMpJlptTUrCCbkDCEE_vDuFb7lEJqxhlaInPThz5ynt_8Hmfugq5JYiyZivvePTkwT4L7OxfpD21mKWYdvAL135dzEEe1lqRG#favicon), however a quick look at some big properties shows sizes bigger than that but still much smaller than our default:\r\n\r\nSite | Icon size\r\n--- | ---\r\ngoogle.com | 5.3 KB\r\nmicrosoft.com | 17.2 KB\r\nstackoverflow.com | 5.5 KB\r\nour template | 31.2 KB\r\n\r\nCracking open the Google favicon.ico reveals they have only two sizes embedded: 32x32@32-bit and 16x16@32-bit. Our default has two more sizes beyond that: 48x48 and 64x64.\r\n\r\nReducing ours to match Google's (seems like a reasonable default) drops the size to match theirs (as expected) at 5.3 KB so we should likely just do that.\r\n\r\nThe smaller, optimized ICO file is in this ZIP: [favicon.zip](https://github.com/aspnet/AspNetCore/files/3548220/favicon.zip)\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/aspnet/AspNetCore/issues/comments/525512946","html_url":"https://github.com/aspnet/AspNetCore/issues/13491#issuecomment-525512946","issue_url":"https://api.github.com/repos/aspnet/AspNetCore/issues/13491","id":525512946,"node_id":"MDEyOklzc3VlQ29tbWVudDUyNTUxMjk0Ng==","user":{"login":"dlemstra","id":10426229,"node_id":"MDQ6VXNlcjEwNDI2MjI5","avatar_url":"https://avatars3.githubusercontent.com/u/10426229?v=4","gravatar_id":"","url":"https://api.github.com/users/dlemstra","html_url":"https://github.com/dlemstra","followers_url":"https://api.github.com/users/dlemstra/followers","following_url":"https://api.github.com/users/dlemstra/following{/other_user}","gists_url":"https://api.github.com/users/dlemstra/gists{/gist_id}","starred_url":"https://api.github.com/users/dlemstra/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dlemstra/subscriptions","organizations_url":"https://api.github.com/users/dlemstra/orgs","repos_url":"https://api.github.com/users/dlemstra/repos","events_url":"https://api.github.com/users/dlemstra/events{/privacy}","received_events_url":"https://api.github.com/users/dlemstra/received_events","type":"User","site_admin":false},"created_at":"2019-08-27T22:49:09Z","updated_at":"2019-08-27T22:49:09Z","author_association":"NONE","body":"An extra optimization would be to change the semi transparent pixels and make them either fully opaque or fully transparent. And together with a reduction of the number of colors would allow the use of a color pallete instead. If this file is optimized using that technique the size can be reduced to 3.5 KB. So you might want to consider to use this file instead: [new-favicon.zip](https://github.com/aspnet/AspNetCore/files/3548369/new-favicon.zip). With this technique there is a small loss in quality but I am really unable to see the difference here."}}
{"actor":{"display_login":"dlemstra"}}