Skip to content

logo.svg in header/footer; outlined favicon + full icon set - #81

Merged
ralyodio merged 1 commit into
masterfrom
feat/brand-icons
Oct 6, 2026
Merged

ralyodio merged 1 commit into
masterfrom
feat/brand-icons

Conversation

@ralyodio

@ralyodio ralyodio commented Oct 6, 2026

Copy link
Copy Markdown
Contributor
  • Header and footer now show logo.svg, the outlined Geist Black wordmark, instead of styled text.
  • favicon.svg was live text in "Arial Black", which most machines don't have. It's now the "c0" mark drawn as outlines on the orange tile.
  • Icon set, generated from it with cli-tools favicon: app/favicon.ico (16 and 32), favicon-16.png, favicon-32.png, apple-touch-icon.png (180), and icon-192.png / icon-512.png. maskable-192.png and maskable-512.png come from a separate full-bleed artwork with the mark inside the centre 80%, so Android's circle mask never clips it.
  • Manifest fix: it pointed at /icon-192.png and /icon-512.png, which didn't exist and returned 404. They exist now, tagged any and maskable.
  • Metadata: SVG, PNG and Apple icons. The forum at /bbs already uses /favicon.svg, so it picks up the new mark too.

Typecheck is clean.

🤖 Generated with Claude Code

- Header and footer show logo.svg (the outlined Geist Black wordmark) instead
  of styled text.
- favicon.svg was live text in 'Arial Black'; it is now the c0 mark as
  outlines on the orange tile. From it, cli-tools 'favicon' generated
  favicon.ico (app/), favicon-16/32, apple-touch-icon (180), icon-192/512, and
  maskable-192/512 from a separate full-bleed artwork.
- manifest icons pointed at /icon-192.png and /icon-512.png, which did not
  exist (404); they do now, with any/maskable purposes.

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

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown

ThreatCrush Security Scan

32 finding(s)

MEDIUM: 29 | LOW: 3

Severity Rule Location
MEDIUM sql-template-interpolation apps/web/app/api/bounties/[id]/claim/route.ts:18
MEDIUM sql-template-interpolation apps/web/app/api/bounties/[id]/claim/route.ts:34
MEDIUM sql-template-interpolation apps/web/app/api/bounties/[id]/claim/route.ts:43
MEDIUM sql-template-interpolation apps/web/app/api/bounties/route.ts:81
MEDIUM sql-template-interpolation apps/web/app/api/coupons/route.ts:68
MEDIUM sql-template-interpolation apps/web/app/api/coupons/route.ts:75
MEDIUM sql-template-interpolation apps/web/app/api/coupons/vote/route.ts:31
MEDIUM sql-template-interpolation apps/web/app/api/coupons/vote/route.ts:33
MEDIUM sql-template-interpolation apps/web/app/api/stores/[slug]/route.ts:9
MEDIUM sql-template-interpolation apps/web/app/api/webhooks/crawlproof/route.ts:83
MEDIUM sql-template-interpolation apps/web/app/api/webhooks/crawlproof/route.ts:90
MEDIUM js-open-redirect apps/web/app/bounties/new/page.tsx:57
MEDIUM js-unescaped-html-sink apps/web/app/coupons/[id]/page.tsx:57
MEDIUM js-unescaped-html-sink apps/web/app/layout.tsx:88
MEDIUM sql-template-interpolation apps/web/app/stores/[slug]/page.tsx:15
MEDIUM sql-template-interpolation apps/web/lib/flipp-sync.ts:445
MEDIUM sql-template-interpolation apps/web/lib/inbound-email.ts:570
MEDIUM sql-template-interpolation apps/web/lib/mcp-tools.ts:81
MEDIUM sql-template-interpolation apps/web/lib/members.ts:54
MEDIUM sql-template-interpolation apps/web/lib/nichedb-sync.ts:186
MEDIUM sql-template-interpolation apps/web/lib/nichedb-sync.ts:206
MEDIUM sql-template-interpolation apps/web/lib/reddit-sync.ts:336
MEDIUM sql-template-interpolation apps/web/lib/reddit-sync.ts:355
MEDIUM sql-template-interpolation apps/web/lib/reddit-sync.ts:361
MEDIUM sql-template-interpolation apps/web/lib/reveal-coupon.ts:115
MEDIUM sql-template-interpolation apps/web/lib/reveal-coupon.ts:170
MEDIUM sh-predictable-temp-path apps/web/public/install.sh:37
MEDIUM sh-predictable-temp-path apps/web/public/install.sh:39
MEDIUM sql-template-interpolation apps/web/scripts/migrate.mjs:192
LOW sql-template-interpolation test/inbound-email.test.mjs:139
LOW sql-template-interpolation test/reveal-sweep.test.mjs:69
LOW sql-template-interpolation test/reveal-sweep.test.mjs:112

Snippets are redacted; ThreatCrush never prints matched credential material.

@ralyodio
ralyodio merged commit 2b3998f into master Oct 6, 2026
5 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