AI_Assistant/pcbjam
Watch
1
0
Fork
You've already forked pcbjam
 0
Code Issues Pull requests Projects Releases 4 Packages Wiki Activity Actions 2

feat(site): add favicon.ico + apple-touch-icon so the tab icon renders

Browse source 
Chrome also fires an implicit request for /favicon.ico; with only an SVG
link declared and no .ico at the site root, that 404 left the tab without
an icon. Add a square multi-res .ico (48/32/16) and a 180x180
apple-touch-icon — both derived from the jar art already embedded in
favicon.svg — and declare the full icon set (ico + svg + apple-touch) in
BaseLayout.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Viktor Vaczi 2026-06-10 17:24:31 +02:00
commit 3978bf5ff1
3 changed files with 5 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

BIN
site/public/apple-touch-icon.png Normal file
View file

Binary file not shown.
Side by side

After

Width:  |  Height:  |  Size: 32 KiB

BIN
site/public/favicon.ico Normal file
View file

Binary file not shown.
Side by side

After

Width:  |  Height:  |  Size: 15 KiB

5
site/src/layouts/BaseLayout.astro
View file

@ -33,7 +33,12 @@ const canonical = Astro.url.href;
<head>
<meta charset="utf-8" />
<meta name="viewport" content="width=device-width, initial-scale=1" />
{/* Favicons: .ico is the universal fallback (also satisfies Chrome's
implicit /favicon.ico request); SVG is preferred by modern browsers;
apple-touch-icon covers the iOS home screen. */}
<link rel="icon" href="/favicon.ico" sizes="32x32" />
<link rel="icon" type="image/svg+xml" href="/favicon.svg" />
<link rel="apple-touch-icon" href="/apple-touch-icon.png" />
<link rel="canonical" href={canonical} />
<meta name="description" content={description} />
<title>{fullTitle}</title>