⚡ Fix favicon: use static PNG instead of broken dynamic SVG #59
Conversation
The icon.tsx was recreating the logo SVG inline with transform attributes that don't render correctly at small sizes in Chrome tabs. Replaced with a static icon.png in src/app/ which Next.js will use automatically as the favicon.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| src="/icon.png" | ||
| alt="Glance" | ||
| className="h-8 w-8" | ||
| /> |
There was a problem hiding this comment.
Logo image references non-existent public file path
Medium Severity
The <img src="/icon.png"> element references a file at the URL path /icon.png, which requires the file to exist in the public/ directory. However, the PR description states the icon.png is being placed in src/app/ for favicon purposes. Files in src/app/ are served by Next.js's file-based metadata conventions (for favicons) but are not accessible at root URLs for <img> elements. The public/ directory currently contains icon.svg but no icon.png, so this image will fail to load with a 404 error.
1 participant
Problem
The
icon.tsxwas recreating the logo SVG inline withtransformattributes that don't render correctly at small sizes in Chrome tabs.Solution
Replace with a static
icon.pnginsrc/app/which Next.js will automatically use as the favicon.Before/After
Before: Broken/clipped favicon in Chrome tabs
After: Clean PNG favicon that renders correctly at all sizes
Note
Low Risk
Simple asset/UI change that removes an edge-rendered icon generator; main risk is missing/incorrect
icon.pngplacement causing a broken favicon/logo.Overview
Stops generating the app icon via
src/app/icon.tsx(NextImageResponseSVG) and instead relies on a staticicon.pngasset for more reliable favicon rendering.Updates
DashboardHeaderto display the new PNG logo next to the title and tweaks the title typography (weight/tracking).Written by Cursor Bugbot for commit 94bacb3. This will update automatically on new commits. Configure here.