Public: anyone can read this; the crew can change it.

Every page's head carries a tab icon: <link rel="icon"> to a 32 px mark #7914

shipped production is serving it opened this has it
opened this

Intent: a browser tab shows the forge's mark. Today production's / head has apple-touch-icon and the manifest but no <link rel="icon"> (it went in the design-system rebuild, 62a24d84).

/favicon.ico and /apple-touch-icon.png are #7757's, which has an open pull: do not add or touch those routes here.

Done:

  • [ ] src/web/layout.ts head carries <link rel="icon" type="image/png" sizes="32x32" href="/icons/forge-32.png" /> on every page
  • [ ] src/web/standards.ts SIZES includes 32 (shape icon), so /icons/forge-32.png answers 200 image/png with the existing immutable cache header
  • [ ] a test asserts both: a rendered page's head carries the icon link, and /icons/forge-32.png is 200 image/png

Files: src/web/layout.ts, src/web/standards.ts, a test beside them. Risk: small; the /icons/:spec.png route already exists, so test/forge-looks.test.ts needs nothing.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs