-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix fallback favicon contrast issues #5818
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Should i be able to see this in the preview? I still see this @delucis |
Favicons are usually cached pretty aggressively, so perhaps not. Is that Chrome? You might be able to force it with a “force reload” (Ctrl + Shift + R I think? If not, from the View menu when pressing Shift.) |
Couldn't seem to force a reload on my end, but I'll trust you and will report back if it doesn't eventually work! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking forward to this, as I'm affected myself!
Description (required)
Updates our fallback favicon (a
.ico
file used by browsers that don’t support SVG favicons — primarily Safari) to a variant with a background colour.The current fallback favicon uses a transparent background, making the white “A” in the Astro logo more or less invisible in light mode browsers (see #5792). In the main SVG favicon the “A” switches colour automatically based on user colour scheme, but we can’t use the same approach for the static fallback.
I opted for a solid background colour as the cleanest solution here. I also experimented with a dark outline around the logo, but at the small scale used for favicons, this looks pretty ugly.
Here’s a quick before/after side-by-side, which also helpfully demonstrates the contrast issue:
Related issues & labels (optional)