Skip to content

Commit b3188b4

Browse files
authored
Simplify favicons and other web icons (#31000)
* Simplify favicons and other web icons browserconfig.xml seems to have died with Internet Explorer `apple-touch-icon` is awfully documented but seems like larger sizes are now preferred Use PNG for the favicon as things now support it across the board, we could even consider moving to SVG favicons in the future Optimised using oxipng, this is to simplify icon replacement in `element-web-bin` Signed-off-by: Michael Telatynski <[email protected]> * Fix paths Signed-off-by: Michael Telatynski <[email protected]> * Update tests Signed-off-by: Michael Telatynski <[email protected]> * Remove border around favicons Signed-off-by: Michael Telatynski <[email protected]> * Add test Signed-off-by: Michael Telatynski <[email protected]> * Add test Signed-off-by: Michael Telatynski <[email protected]> --------- Signed-off-by: Michael Telatynski <[email protected]>
1 parent 5dc8edc commit b3188b4

35 files changed

+111
-121
lines changed

res/manifest.json

Lines changed: 10 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -5,49 +5,24 @@
55
"theme_color": "#76CFA6",
66
"start_url": "index.html",
77
"icons": [
8-
{
9-
"src": "/vector-icons/44.png",
10-
"sizes": "44x44",
11-
"type": "image/png"
12-
},
13-
{
14-
"src": "/vector-icons/1240x600.png",
15-
"sizes": "1240x600",
16-
"type": "image/png"
17-
},
18-
{
19-
"src": "/vector-icons/300.png",
20-
"sizes": "300x300",
21-
"type": "image/png"
22-
},
23-
{
24-
"src": "/vector-icons/150.png",
25-
"sizes": "150x150",
26-
"type": "image/png"
27-
},
28-
{
29-
"src": "/vector-icons/88.png",
30-
"sizes": "88x88",
31-
"type": "image/png"
32-
},
338
{
349
"src": "/vector-icons/24.png",
3510
"sizes": "24x24",
3611
"type": "image/png"
3712
},
3813
{
39-
"src": "/vector-icons/50.png",
40-
"sizes": "50x50",
14+
"src": "/vector-icons/120.png",
15+
"sizes": "120x120",
4116
"type": "image/png"
4217
},
4318
{
44-
"src": "/vector-icons/620x300.png",
45-
"sizes": "620x300",
19+
"src": "/vector-icons/144.png",
20+
"sizes": "144x144",
4621
"type": "image/png"
4722
},
4823
{
49-
"src": "/vector-icons/1024.png",
50-
"sizes": "1024x1024",
24+
"src": "/vector-icons/152png",
25+
"sizes": "152x152",
5126
"type": "image/png"
5227
},
5328
{
@@ -56,18 +31,13 @@
5631
"type": "image/png"
5732
},
5833
{
59-
"src": "/vector-icons/152.png",
60-
"sizes": "152x152",
61-
"type": "image/png"
62-
},
63-
{
64-
"src": "/vector-icons/120.png",
65-
"sizes": "120x120",
34+
"src": "/vector-icons/512.png",
35+
"sizes": "512x512",
6636
"type": "image/png"
6737
},
6838
{
69-
"src": "/vector-icons/76.png",
70-
"sizes": "76x76",
39+
"src": "/vector-icons/1024.png",
40+
"sizes": "1024x1024",
7141
"type": "image/png"
7242
}
7343
],

res/vector-icons/1024.png

-22.3 KB
Loading

res/vector-icons/120.png

1.16 KB
Loading

res/vector-icons/1240x600.png

-21.9 KB
Binary file not shown.

res/vector-icons/144.png

6.08 KB
Loading

res/vector-icons/150.png

-4.7 KB
Binary file not shown.

res/vector-icons/152.png

2.06 KB
Loading

res/vector-icons/180.png

2.27 KB
Loading

res/vector-icons/24.png

-52 Bytes
Loading

res/vector-icons/300.png

-9.14 KB
Binary file not shown.

0 commit comments

Comments
 (0)