Skip to content

fix: favicon non servi + séparateur de titre |#10

Merged
Alexis-Gontier merged 1 commit into
mainfrom
fix/favicon-and-title-separator
Jun 21, 2026
Merged

fix: favicon non servi + séparateur de titre |#10
Alexis-Gontier merged 1 commit into
mainfrom
fix/favicon-and-title-separator

Conversation

@Alexis-Gontier

Copy link
Copy Markdown
Owner

Problèmes corrigés

1. Le favicon ne s'affichait pas

Le matcher du proxy next-intl (proxy.ts) n'excluait que les chemins avec extension. La route metadata racine app/icon.tsx (/icon, sans extension) était donc 307-redirigée dans [locale]/fr/icon renvoyait du HTML au lieu du PNG, et le navigateur n'obtenait jamais l'icône.

→ Ajout de icon à la liste d'exclusion du matcher. /icon sert désormais directement un PNG 32×32 (HTTP 200, image/png).

2. Séparateur du titre

Template de titre passé de %s · … à %s | … (ex. Projets | Alexis Gontier).

Tests

  • ✅ Biome, typecheck, Vitest (7 tests) — hooks pre-commit/pre-push
  • ✅ Vérifié manuellement : /icon → HTTP 200 image/png (519 o, PNG 32×32)

- exclude `/icon` from the next-intl proxy matcher so the root metadata
  route is no longer 307-redirected into [locale] (favicon now resolves)
- switch the page title template separator from ` · ` to ` | `
@vercel

vercel Bot commented Jun 20, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
algont-ier-com Ready Ready Preview, Comment Jun 20, 2026 4:06am

@Alexis-Gontier
Alexis-Gontier merged commit 96d3b53 into main Jun 21, 2026
5 checks passed
@Alexis-Gontier
Alexis-Gontier deleted the fix/favicon-and-title-separator branch June 21, 2026 20:40
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