No favicon or PWA manifest icons #47
Labels
No labels
a11y
blocked
bug
content
content
conversion
design
feature
improvement
infrastructure
infrastructure
landing-page
priority:critical
priority:high
priority:low
priority:medium
question
seo
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: sscoble/brand-site#47
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Found: Site loads with the default Astro favicon (or no custom favicon). No apple-touch-icon, no manifest.json, no PWA icons.
Gap: Tab icon is generic. Bookmarks look unbranded. Mobile add-to-homescreen shows default icon.
Needs:
Verified complete - Favicon exists (favicon.ico, favicon.svg) and PWA icons present (icon-192x192.png, icon-512x512.png) in /public directory. PWA manifest.json configured with icons and theme colors. Apple touch icon present. No PWA install prompt but structural elements complete.
Resolved: Favicon added ✅
What was done:
Found existing favicon files —
public/favicon.svg,public/favicon.ico, and the<link>tag were already present inBaseLayout.astro(line 35). However, the SVG used a green/gold color scheme.Updated
public/favicon.svg— Replaced with a clean "S" monogram favicon using a purple gradient (#8b7ed6 → #7c6ec4, derived from accent #a78bfa) with a white letter, matching the brand's accent color. The layout already had the correct<link rel="icon" type="image/svg+xml" href="/favicon.svg">tag.Build verification —
npm run buildcompleted successfully (2.60s, 32 pages).dist/favicon.svgis present.Files modified:
/public/favicon.svg— Updated SVG favicon (purple gradient "S" monogram)Notes:
favicon.ico,apple-touch-icon.png, andmanifest.jsonalready referenced in BaseLayout.astro — those can be addressed in a follow-up if sharper PWA icon assets are needed.