Skip to content

Use transparent logo in header + favicon#46

Merged
polter-dev merged 1 commit into
mainfrom
polish/transparent-logo
Jul 12, 2026
Merged

Use transparent logo in header + favicon#46
polter-dev merged 1 commit into
mainfrom
polish/transparent-logo

Conversation

@polter-dev

Copy link
Copy Markdown
Collaborator

Summary

  • replace the header mark and app icon with the provided transparent 512x512 PNG
  • increase the shared header mark from 24px to 28px while preserving square, contained rendering and decorative semantics
  • update the existing Playwright size assertion

TDD evidence

  • Red: npx playwright test e2e/feed.spec.ts --grep "verified feed" --project=chromium-desktop failed because the mark still measured 24px
  • Green: the same focused test passed after the 28px CSS change

Verification

  • npm run lint
  • npm run typecheck
  • npm run test:unit (311 passed, 6 skipped)
  • npm run test:e2e (36 passed, 2 skipped)
  • fixture dev routes /, /analyze, and /feed/feed-item-treasury-bills returned the shared decorative header mark
  • served /icon.png byte-matches src/app/icon.png; no favicon.ico is present

Manual QA note

The in-app browser backend was unavailable in this environment, so interactive tab-level visual inspection could not be completed. Real desktop/mobile Chromium coverage, direct rendered-route inspection, and PNG alpha/icon-response checks passed.

@vercel

vercel Bot commented Jul 12, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
capcheck Ready Ready Preview, Comment Jul 12, 2026 1:25am

@polter-dev
polter-dev merged commit efbf7d7 into main Jul 12, 2026
4 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