swap the astro default favicon for a brand-matched shield#20
Conversation
…b matches the brand: live deploy was rendering the astro logo (the dark-A glyph) on every browser tab because the default web/public/favicon.svg ships with the astro project starter and the layout never linked the file explicitly so browsers fell back to whatever favicon.ico happened to exist (also the astro default), which means every tab + bookmark + history entry showed the unrelated astro mark instead of a shield icon for an app called chainshield; replaced web/public/favicon.svg with a heater-shield silhouette built around the project palette (paper #f4ede2 fill on dark tab themes, ink #1a1a1a on light themes via prefers-color-scheme media query embedded in the svg style block, lime #c4ff5b accent for the inner check stroke matching --accent in global.css), proportions tuned to read at 16px (single bold check stroke, no fine internal detail that would smudge in a tab list); added an explicit <link rel="icon" type="image/svg+xml" href="/favicon.svg"> to web/src/layouts/Layout.astro so modern browsers pick up the SVG directly without depending on the auto-discovery fallback chain; deleted web/public/favicon.ico because keeping the legacy astro-shipped raster around as a fallback for very old browsers would just reintroduce the wrong logo on the few clients that prefer .ico over the typed link, and the explicit svg link covers every browser shipped this decade
|
Warning Rate limit exceeded
To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Review rate limit: 0/1 reviews remaining, refill in 32 minutes and 39 seconds.Comment |
Deploying chainshield with
|
| Latest commit: |
26cf678
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://4f730f2a.chainshield.pages.dev |
| Branch Preview URL: | https://feat-shield-favicon.chainshield.pages.dev |
1 participant
Summary
favicon.svg(a dark "A" glyph) with a heater-shield silhouette using the project palette (#1a1a1aink /#f4ede2paper viaprefers-color-scheme,#c4ff5blime check accent matching--accentinglobal.css).<link rel="icon" type="image/svg+xml" href="/favicon.svg" />intoweb/src/layouts/Layout.astroso browsers don't depend on auto-discovery.web/public/favicon.ico(the Astro-default raster) so legacy.ico-preferring clients don't reintroduce the unrelated logo as a fallback.Tab now shows a shield + lime check on hover, regardless of whether the tab strip renders dark or light.
Test plan
bun run typecheck:web— 0 errors / warnings / hintsbun run build:web— cleangrep favicon web/dist/index.htmlconfirms the rendered HTML now carries<link rel="icon" type="image/svg+xml" href="/favicon.svg">Need help on this PR? Tag
@codesmithwith what you need.