Skip to content

site: serve the favicon by URL so Google shows the logo in search results - #691

Merged
chaitanyagiri merged 1 commit into
mainfrom
site/favicon-for-google
Oct 4, 2026
Merged

chaitanyagiri merged 1 commit into
mainfrom
site/favicon-for-google

Conversation

@chaitanyagiri

Copy link
Copy Markdown
Collaborator

Why the logo does not show next to munderdiffl.in in Google results, and the fix. Website files only.

Cause

  • The home page set its icon as an embedded data: URI. Google only uses an icon it can fetch by URL.
  • https://munderdiffl.in/favicon.ico returned 404.

Change

  • docs/index.html: the embedded icon is replaced by links to /logo.svg, /favicon-48.png, /favicon-192.png, /favicon.ico and /apple-touch-icon.png.
  • New files: docs/favicon-48.png, docs/favicon-192.png (Google asks for a multiple of 48px) and docs/favicon.ico, all made from the existing docs/logo.png.
  • Nothing else on the page changes.

After merge

Google picks up a favicon on its next crawl of the home page, which can take days to a few weeks. I will request a recrawl of the home page in Search Console.

Before

before

After

after

🤖 Generated with Claude Code

…results

The home page embedded its icon as a data: URI and /favicon.ico returned 404.
Google only uses an icon it can fetch, so results showed a globe. Link
/logo.svg, 48px and 192px PNGs and /favicon.ico instead.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

🚫 This PR is missing its before/after evidence

Every pull request here has to show its work. Screenshots or a short screen recording, before the change and after it.

  • After — no image or video under that heading

How to fix it: edit the description, keep the ### Before and ### After headings from the template, and drag an image or video under each. GitHub uploads it inline. This check re-runs the moment you save.

A bug fix with no visible surface still needs it: show the failing behaviour, then the same steps passing. A terminal recording is fine.

Genuinely nothing to show — a CI tweak, a typo, a dependency bump? A maintainer can apply the no-visual-change label. Please don't ask unless it truly has no observable effect.

📖 CONTRIBUTING.md → Evidence is mandatory

@chaitanyagiri
chaitanyagiri merged commit 71dbbc1 into main Oct 4, 2026
2 of 3 checks passed
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