Skip to content

Marketing: add favicon.ico + icon set for Google Search - #5

Merged
angesanze merged 1 commit into
mainfrom
production-readiness
Jul 6, 2026
Merged

Marketing: add favicon.ico + icon set for Google Search#5
angesanze merged 1 commit into
mainfrom
production-readiness

Conversation

@angesanze

Copy link
Copy Markdown
Owner

Googlebot cerca /favicon.ico alla radice: era 404 su varasto.rocks. Aggiunge favicon.ico, PNG 192/512, web manifest e i relativi . Già live e verificato (tutti 200); allinea main.

🤖 Generated with Claude Code

Googlebot auto-fetches /favicon.ico, which was 404 on varasto.rocks (only an
SVG icon was linked). Add the real favicon.ico, 192/512 PNGs and a web
manifest, and reference them all in <head>.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@angesanze
angesanze merged commit e5b1e16 into main Jul 6, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a web manifest file and updates the layout template to link the manifest, a fallback favicon, and high-resolution icons. The review feedback suggests optimizing icon loading by adding a sizes attribute to the fallback favicon to prevent duplicate downloads and removing redundant PNG icon links from the HTML since they are already declared in the manifest.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

<head>
<meta charset="utf-8" />
<meta name="viewport" content="width=device-width, initial-scale=1" />
<link rel="icon" href="/favicon.ico" />

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Per evitare che alcuni browser (come Chrome) scarichino sia il file .ico che il file .svg contemporaneamente, è consigliabile specificare l'attributo sizes="32x32" (o 48x48) sul tag <link> di favicon.ico. Questo indica chiaramente al browser la dimensione dell'icona fallback, permettendogli di preferire l'icona SVG ad alta risoluzione quando supportata.

    <link rel="icon" href="/favicon.ico" sizes="32x32" />

Comment on lines +22 to +24
<link rel="icon" type="image/png" sizes="192x192" href="/icon-192.png" />
<link rel="icon" type="image/png" sizes="512x512" href="/icon-512.png" />
<link rel="apple-touch-icon" href="/icon-192.png" />

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

I tag <link> per le icone PNG da 192x192 e 512x512 sono ridondanti nell'HTML. I browser moderni utilizzeranno l'icona SVG (/varasto.svg), mentre i dispositivi mobili e le PWA che necessitano delle icone PNG ad alta risoluzione le leggeranno direttamente dal file di manifest (/site.webmanifest), che è già collegato alla riga 25. Dichiarare queste icone PNG di grandi dimensioni direttamente nell'HTML può causare download non necessari da parte di alcuni browser, rallentando il caricamento della pagina. Si consiglia di rimuoverle.

    <link rel="apple-touch-icon" href="/icon-192.png" />

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant