Skip to content

feat: branded PRison favicon#62

Merged
mfozmen merged 1 commit into
mainfrom
feat/branded-favicon
Jul 1, 2026
Merged

feat: branded PRison favicon#62
mfozmen merged 1 commit into
mainfrom
feat/branded-favicon

Conversation

@mfozmen

@mfozmen mfozmen commented Jul 1, 2026

Copy link
Copy Markdown
Owner

Replace the default Next.js favicon with an on-brand jail-window mark (accent-green rounded square + white bars, matching the header logo). Shipped as app/icon.svg (crisp modern SVG favicon) + a regenerated multi-size app/favicon.ico (16/32/48).

🤖 Generated with Claude Code

Replace the default create-next-app favicon with an on-brand mark: an accent-
green rounded square with white jail-window bars — a nod to "PRison", matching
the header logo's green square. Ship it as `app/icon.svg` (crisp SVG favicon for
modern browsers) plus a regenerated multi-size `app/favicon.ico` (16/32/48) for
legacy /favicon.ico requests.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@sonarqubecloud

sonarqubecloud Bot commented Jul 1, 2026

Copy link
Copy Markdown

@mfozmen mfozmen merged commit bdecf00 into main Jul 1, 2026
3 of 4 checks passed
@mfozmen mfozmen deleted the feat/branded-favicon branch July 1, 2026 21:03
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

Advisory Review

Scope: app/icon.svg (new, 10-line static SVG) + app/favicon.ico (regenerated binary, 16/32/48).

No bugs, security issues, or convention violations found — this is a pure static-asset swap with no logic changes, so TDD/test-coverage concerns don't apply here. app/icon.svg correctly uses the Next.js App Router icon file convention (auto-detected alongside favicon.ico as a fallback). SVG markup is well-formed and minimal.

Nothing blocking. No inline comments filed.

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