fix: add favicon.ico fallbacks to all examples#15262
Merged
matthewp merged 1 commit intowithastro:mainfrom Jan 21, 2026
Merged
Conversation
🦋 Changeset detectedLatest commit: 0d29731 The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
matthewp
approved these changes
Jan 20, 2026
TheOtterlord
requested changes
Jan 20, 2026
Member
There was a problem hiding this comment.
Thanks for the PR @CloudyWSA
These ico files use our old logo. The assets for our up-to-date logo can be found at https://astro.build/press/. We don't have an ico, but it's downloadable as both an SVG & PNG so can be used to create an ico file.
CloudyWSA
force-pushed
the
fix/favicon-search-results
branch
from
edb3a38 to
3b11461
Compare
Contributor
Author
|
Thanks for the review! I’ve updated the favicons to use the current logo (white flame) and pushed the update. |
Contributor
CloudyWSA
force-pushed
the
fix/favicon-search-results
branch
from
3b11461 to
0d29731
Compare
Contributor
Author
|
Sorry about the wrong images, already updated to the correct ones. Thanks! |
Merged
florian-lefebvre
added a commit
that referenced
this pull request
Jan 22, 2026
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Emanuele Stoppa <[email protected]> Co-authored-by: Matt Kane <[email protected]> Co-authored-by: Matthew Phillips <[email protected]> Co-authored-by: Matthew Phillips <[email protected]> Co-authored-by: Florian Lefebvre <[email protected]> Co-authored-by: Erika <[email protected]> Co-authored-by: Sarah Rainsberger <[email protected]> Co-authored-by: Armand Philippot <[email protected]> Co-authored-by: matthewp <[email protected]> Co-authored-by: florian-lefebvre <[email protected]> Co-authored-by: ematipico <[email protected]> Co-authored-by: Luiz Ferraz <[email protected]> Co-authored-by: HiDeoo <[email protected]> Co-authored-by: Princesseuh <[email protected]> Co-authored-by: Chris Swithinbank <[email protected]> Co-authored-by: Antony Faris <[email protected]> Co-authored-by: Erika <[email protected]> Co-authored-by: Houston (Bot) <[email protected]> Co-authored-by: Florian Lefebvre <[email protected]> Co-authored-by: Volpeon <[email protected]> Co-authored-by: Martin Trapp <[email protected]> Co-authored-by: fkatsuhiro <[email protected]> Co-authored-by: Oliver Speir <[email protected]> Co-authored-by: Andreas Deininger <[email protected]> Co-authored-by: Roman <[email protected]> Co-authored-by: fabon <[email protected]> Co-authored-by: Raanelom <[email protected]> Co-authored-by: Rahul Dogra <[email protected]> Co-authored-by: James Garbutt <[email protected]> Co-authored-by: Pegasus <[email protected]> Co-authored-by: Cameron Smith <[email protected]> Co-authored-by: Martin Trapp <[email protected]> Co-authored-by: Rafael ヤスヒデ 須藤 <[email protected]> Co-authored-by: Matthew Phillips <[email protected]> Co-authored-by: Julien Cayzac <[email protected]> Co-authored-by: Drew Powers <[email protected]> Co-authored-by: Emanuele Stoppa <[email protected]> Co-authored-by: MkDev11 <[email protected]> Co-authored-by: andy <[email protected]> Co-authored-by: Luky Setiawan <[email protected]> Co-authored-by: btea <[email protected]> Co-authored-by: cid <[email protected]> Co-authored-by: Simen Sagholen Førrisdal <[email protected]> Co-authored-by: Alex Launi <[email protected]> Co-authored-by: Kedar Vartak <[email protected]> Co-authored-by: yy <[email protected]> Co-authored-by: Matheus Baroni <[email protected]> Co-authored-by: Kevin Brown <[email protected]> fix(ci): Reinstall deps after having published VS Code (#14996) fix(svelte): allow client directives (#15004) fix(assets): Fixes missing format option for svgs in the passthrough service (#14987) fix(deps): update all non-major dependencies (#15020) fix(content-layer): Try a smarter solution to normalize bare image paths in JSON (#15028) fix(astro): assets vite build log (#15034) resolved (#15033) fix: Remote images: Prevent internal caching from interfering with Astro's cache (#14954) fix(deps): update astro adapters (#15084) fix(deps): update all non-major dependencies (#15072) fix(deps): update astro client runtimes (#15085) fix: move ts-plugin node_modules to dist (#15083) fix: components imports paths (#15107) fix(assets): Use Vite's isFileLoadingAllowed to check if a file can be loaded (#15052) fix(vscode): Correctly handle TypeScript blocks ending with types (#15109) fix(svelte): improve Svelte children prop type checking (#15070) fix Firefox e2e tests for playwright 1.57 (#15113) fix(deps): update astro dependencies (#15103) fix: lint vt test (#15114) fix(deps): update language tools (#15104) resolver abstraction (#15111) fix(vue): add HTML attributes to generated TypeScript types (#15016) Fixes #14686 fix(vscode): Don't update @types/vscode automatically (#15131) fix: adjust page warning to only show up in more relevant times (#15127) fix(dev): preserve query params when base path is stripped (#15124) fix(assets): hoist ?? inside JSON.stringify in virtual module codegen (#15140) fix(toolbar): skip image audit for framework components (#15149) Fixes #15048 fix: Accept setCookie from both context and headers (#15152) fix(ci): Move ts-plugin-bundle to node_modules after recreating node_modules (#15156) fix: scripts not rendering with unused Fragment slots (#13847) (#15147) Fix hydration for framework components in MDX slots (#15150) fix(deps): update astro adapters (#15173) fix: fix image 500 error when moving dist directory in standalone Node (#15169) fix: Allow node: prefix for Node builtins for Vercel middleware (#14863) fix: Allow node: prefix for Node builtins for Vercel middleware (#14839)" fix(node): hash URL stripping (#15196) fix(core): add defensive validation for mod.page in App.render (#15148) fix(css): rewrite absolute URLs with base path in dev mode (#14622) Closes #14585 fix(docs): replace outdated Astro docs links (#15199) fix(lint): Move ESLint comment (#15216) fix: update devalue to the latest (#15222) fix(css): prevent CSS double-bundling (#14991) (#15017) Fixes #14991 fix: assets referenced via js chunks are not flagged as orphaned (#14607) fix(build): Prevent duplicate CSS for hydrated client components" (#14612)" Fixes #14252 fix(deps): update astro client runtimes (#15240) fix(deps): update astro adapters (#15243) fix(deps): update astro dependencies (#15241) fix(deps): update dependency prettier to ^3.8.0 (#15244) fix(deps): update all non-major dependencies (#15242) Fix greedy regex in error message markdown rendering (#15230) Fixes #15068 fixes: #15252 fix: prevent font copying when stopping dev server with q+enter (#15178) fix: renovate config typo (#15256) Fixes #15251 fix: typo in comment (#15232) fix(deps): update dependency svelte to v5.46.4 [security] (#15220) fix: add favicon.ico fallbacks to all examples (#15262)
florian-lefebvre
added a commit
that referenced
this pull request
Jan 30, 2026
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Emanuele Stoppa <[email protected]> Co-authored-by: Matt Kane <[email protected]> Co-authored-by: Matthew Phillips <[email protected]> Co-authored-by: Matthew Phillips <[email protected]> Co-authored-by: Florian Lefebvre <[email protected]> Co-authored-by: Erika <[email protected]> Co-authored-by: Sarah Rainsberger <[email protected]> Co-authored-by: Armand Philippot <[email protected]> Co-authored-by: matthewp <[email protected]> Co-authored-by: florian-lefebvre <[email protected]> Co-authored-by: ematipico <[email protected]> Co-authored-by: Luiz Ferraz <[email protected]> Co-authored-by: HiDeoo <[email protected]> Co-authored-by: Princesseuh <[email protected]> Co-authored-by: Chris Swithinbank <[email protected]> Co-authored-by: Claude Opus 4.5 <[email protected]> Co-authored-by: Antony Faris <[email protected]> Co-authored-by: Erika <[email protected]> Co-authored-by: Houston (Bot) <[email protected]> Co-authored-by: Florian Lefebvre <[email protected]> Co-authored-by: Volpeon <[email protected]> Co-authored-by: Martin Trapp <[email protected]> Co-authored-by: fkatsuhiro <[email protected]> Co-authored-by: Oliver Speir <[email protected]> Co-authored-by: Andreas Deininger <[email protected]> Co-authored-by: Roman <[email protected]> Co-authored-by: fabon <[email protected]> Co-authored-by: Raanelom <[email protected]> Co-authored-by: Rahul Dogra <[email protected]> Co-authored-by: James Garbutt <[email protected]> Co-authored-by: Pegasus <[email protected]> Co-authored-by: Cameron Smith <[email protected]> Co-authored-by: Martin Trapp <[email protected]> Co-authored-by: Rafael ヤスヒデ 須藤 <[email protected]> Co-authored-by: Matthew Phillips <[email protected]> Co-authored-by: Julien Cayzac <[email protected]> Co-authored-by: Drew Powers <[email protected]> Co-authored-by: Emanuele Stoppa <[email protected]> Co-authored-by: MkDev11 <[email protected]> Co-authored-by: andy <[email protected]> Co-authored-by: Luky Setiawan <[email protected]> Co-authored-by: btea <[email protected]> Co-authored-by: cid <[email protected]> Co-authored-by: Simen Sagholen Førrisdal <[email protected]> Co-authored-by: Alex Launi <[email protected]> Co-authored-by: Kedar Vartak <[email protected]> Co-authored-by: yy <[email protected]> Co-authored-by: Matheus Baroni <[email protected]> Co-authored-by: Kevin Brown <[email protected]> Co-authored-by: Eric Grill <[email protected]> Co-authored-by: Cameron Pak <[email protected]> Co-authored-by: Tony Narlock <[email protected]> Co-authored-by: John Mortlock <[email protected]> Co-authored-by: Patrick Arlt <[email protected]> Co-authored-by: Mark Ignacio <[email protected]> fix(ci): Reinstall deps after having published VS Code (#14996) fix(svelte): allow client directives (#15004) fix(assets): Fixes missing format option for svgs in the passthrough service (#14987) fix(deps): update all non-major dependencies (#15020) fix(content-layer): Try a smarter solution to normalize bare image paths in JSON (#15028) fix(astro): assets vite build log (#15034) resolved (#15033) fix: Remote images: Prevent internal caching from interfering with Astro's cache (#14954) fix(deps): update astro adapters (#15084) fix(deps): update all non-major dependencies (#15072) fix(deps): update astro client runtimes (#15085) fix: move ts-plugin node_modules to dist (#15083) fix: components imports paths (#15107) fix(assets): Use Vite's isFileLoadingAllowed to check if a file can be loaded (#15052) fix(vscode): Correctly handle TypeScript blocks ending with types (#15109) fix(svelte): improve Svelte children prop type checking (#15070) fix Firefox e2e tests for playwright 1.57 (#15113) fix(deps): update astro dependencies (#15103) fix: lint vt test (#15114) fix(deps): update language tools (#15104) resolver abstraction (#15111) fix(vue): add HTML attributes to generated TypeScript types (#15016) Fixes #14686 fix(vscode): Don't update @types/vscode automatically (#15131) fix: adjust page warning to only show up in more relevant times (#15127) fix(dev): preserve query params when base path is stripped (#15124) fix(assets): hoist ?? inside JSON.stringify in virtual module codegen (#15140) fix(toolbar): skip image audit for framework components (#15149) Fixes #15048 fix: Accept setCookie from both context and headers (#15152) fix(ci): Move ts-plugin-bundle to node_modules after recreating node_modules (#15156) fix: scripts not rendering with unused Fragment slots (#13847) (#15147) Fix hydration for framework components in MDX slots (#15150) fix(deps): update astro adapters (#15173) fix: fix image 500 error when moving dist directory in standalone Node (#15169) fix: Allow node: prefix for Node builtins for Vercel middleware (#14863) fix: Allow node: prefix for Node builtins for Vercel middleware (#14839)" fix(node): hash URL stripping (#15196) fix(core): add defensive validation for mod.page in App.render (#15148) fix(css): rewrite absolute URLs with base path in dev mode (#14622) Closes #14585 fix(docs): replace outdated Astro docs links (#15199) fix(lint): Move ESLint comment (#15216) fix: update devalue to the latest (#15222) fix(css): prevent CSS double-bundling (#14991) (#15017) Fixes #14991 fix: assets referenced via js chunks are not flagged as orphaned (#14607) fix(build): Prevent duplicate CSS for hydrated client components" (#14612)" Fixes #14252 fix(deps): update astro client runtimes (#15240) fix(deps): update astro adapters (#15243) fix(deps): update astro dependencies (#15241) fix(deps): update dependency prettier to ^3.8.0 (#15244) fix(deps): update all non-major dependencies (#15242) Fix greedy regex in error message markdown rendering (#15230) Fixes #15068 fixes: #15252 fix: prevent font copying when stopping dev server with q+enter (#15178) fix: renovate config typo (#15256) Fixes #15251 fix: typo in comment (#15232) fix(deps): update dependency svelte to v5.46.4 [security] (#15220) fix: add favicon.ico fallbacks to all examples (#15262) fix(cli): add .vercel to .gitignore when using astro add vercel (#15185) Closes #15058 fix(fonts): font providers as class instances (#15286) Fix: Remove await from getActionResult JSDoc example (#15304) fix(errors): Only style valid URLs in the error overlay (#15324) fix(fonts): remove getFontBuffer() (#15334)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
favicon.icoreference for templates that previously relied on SVG-only favicons.Fixes #15236
Testing
<link rel="icon" href="/favicon.ico">.favicon.icois correctly served from the root when present inpublic/.Docs