Update prefetch.mdx to use more technically correct language (discord username finjo)#14030
Update prefetch.mdx to use more technically correct language (discord username finjo)#14030finleyjb wants to merge 1 commit into
Conversation
Maybe this splitting hairs, but the existing page says that it works for links to the same site. This is not [technically correct](https://web.dev/articles/same-site-same-origin), because `same-site` URLs will work on different subdomains. If this does not work on different subdomains, it's technically `same-origin` rather than `same-site`.
✅ Deploy Preview for astro-docs-2 ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
|
Hello! Thank you for opening your first PR to Astro’s Docs! 🎉 Here’s what will happen next:
|
|
Hi, thanks for taking the time to submit a PR but I'm not sure what this is supposed to fix...
See: https://docs.astro.build/en/guides/prefetch/#browser-support And according to MDN:
So, according to your link... your fix (same-origin) would be wrong? |
|
Oh interesting. You're right.
I was trying it out for a link from my domain welcomehome.city to a subdomain blog.welcomehome.city. Didn't seem to work. Maybe I'll give it another shot.
…On Wednesday, June 10th, 2026 at 4:02 AM, Armand Philippot ***@***.***> wrote:
ArmandPhilippot left a comment [(withastro/docs#14030)](#14030 (comment))
Hi, thanks for taking the time to submit a PR but I'm not sure what this is supposed to fix...
> Astro’s prefetching uses [<link rel="prefetch">](https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes/rel/prefetch) if supported by the browser, and falls back to the [fetch() API](https://developer.mozilla.org/en-US/docs/Web/API/Fetch_API) otherwise.
See: https://docs.astro.build/en/guides/prefetch/#browser-support
And according to [MDN](https://developer.mozilla.org/en-US/docs/Web/HTML/Reference/Attributes/rel/prefetch):
> <link rel="prefetch"> is used for same-site navigation resources, or for subresources used by same-site pages.
So, according to your link... your fix (same-origin) would be wrong?
—
Reply to this email directly, [view it on GitHub](#14030?email_source=notifications&email_token=ARSYH7X3O5HAEV42LFD6VZL47EW37A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINRWHA4TCMBVGMZ2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KYZTPN52GK4S7MNWGSY3L#issuecomment-4668910533), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/ARSYH7UHYHLGWJ22EAAWKOT47EW37AVCNFSNUABFKJSXA33TNF2G64TZHMZTONZTG43TCMRSHNEXG43VMU5TINRSGU2TIMJVGU32C5QC).
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for [iOS](https://github.com/notifications/mobile/ios/ARSYH7XV6R7AK65NBQBJJJT47EW37A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINRWHA4TCMBVGMZ2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KUZTPN52GK4S7NFXXG) and [Android](https://github.com/notifications/mobile/android/ARSYH7SPSINLEJLXRVO54ZD47EW37A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINRWHA4TCMBVGMZ2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2K4ZTPN52GK4S7MFXGI4TPNFSA). Download it today!
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Description
Maybe this splitting hairs, but the existing page says that it works for links to the same site. This is not technically correct, because
same-siteURLs will work on different subdomains. If this does not work on different subdomains, it's technicallysame-originrather thansame-site.