-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Internationalization (i18n) guide and recipe should be merged #9256
Comments
Thank you for this amazingly detailed feedback, @aureliendossantos ! We have a bit of a legacy problem here, and I do have updating i18n API docs on my plate, and probably including better linking to and discovery of that guide. So the backstory is: This recipe was written well before we had the i18n Routing API. It's based on how we created our i18n system for Astro Docs, and then Starlight, both of which are older than our i18n Routing API. (And which have been updated to take a little from, but still do not yet fully use this API for routing) So, the guide is a bit more complete about setting up an i18n site as a whole, because that's what we did! And the i18n Routing guide contains information about only the routing part of it (ignoring any "extra features" like translating UI strings because it's not specific to routing, but it is a huge part of Astro Docs/Starlight implementations). So the current situation is: The two don't perfectly align! And as you've correctly identified, we don't right now have a unified "here's how to set up a site" story. Add to the mix that our experimental Content Layer API is dropping in the 5.0 beta NEXT WEEK, which will probably open up to even more content management options (like pulling in your translations from external sources), and we certainly don't have a full i8n story around that yet to tell! All this is to say that this would be an amazing addition to our current docs that we don't currently provide in an all-in-one format. I would certainly be open to anyone wanting to help contribute to that! It's on our wishlist, but not our immediate plate with lots of new docs currently written to support the new features in and upgrading to v5. BUT, this post will be a super helpful kick off for that, so I hope people see this, continue to leave feedback about what will be helpful, and maybe propose some contributions! |
Just bumping this for freshness in case anyone would like to "freshen up" our i18n guide that pre-dates our i18n routing API features! |
I agree the recipe could be merged with the guide because many expect to find this kind of information there (translating UI strings or routes). However, I see two caveats:
Translate the slug? With:
Considering that the site is available with the
Meanwhile, with:
We can have:
From both a user and SEO perspective, it's better. Perhaps the file names in the example should be translated to make it more meaningful. My example can probably be rewritten using
But it might be more complicated to explain to newcomers... especially without knowing where their content comes from and if they want different page templates. |
To be clear, I think if there's any improvement/solution to be had here, it's in refreshing the recipe and NOT moving the content into the guide page, for similar reasons: a recipe is an opinionated suggestion of a "base dish". Also, the Internationalization page is about using the API which is a routing API only and should stay focused on the connection between creating/validating route path not actual translation. (Which again, will be very much up to an individual project.) The current recipe as it exists works, and I think the question is, can it be made more helpful by acknowledging the existence of the newer routing API? The answer might be "no", but that is the question to be asking here, and the way we will evaluate potential improvements! |
I think the recipe could be simplified somewhat, at least the |
Thanks @sasoria ! Would you be willing to make a PR to the recipe, or help guide us through exactly what's different about your example? It certainly does make sense that if we can incorporate |
Sure, I could try to make a PR this week! |
Thank you, @sasoria ! I know this is one inconsistency (between these two pages) that often gets mentioned! |
📚 Subject area/topic
Internationalization (i18n)
📋 Page(s) affected (or suggested, for new content)
https://docs.astro.build/en/guides/internationalization/
https://docs.astro.build/en/recipes/i18n/
📋 Description of content that is out-of-date or incorrect
The guide Internationalization (i18n) Routing lacks several tips that would make the Astro feature more attractive. Personally, I had given up on using Astro's i18n until I realised the following points, which made my experience so much better:
Every example block shows a directory structure with a folder per locale, in which you have to duplicate every .astro file:
It seems inconvenient and off-putting, I can't imagine a case where I'd like to do this. The docs suggests in the text that you could create a
/[locale]/
folder, but I must admit that when skimming through the page, I missed it and only saw the several examples like in the screenshot. I think a more proeminent example block with/[locale]
and/or/[...locale]
(see 2.) would better showcase the flexibility of the feature.An info is missing from the guide and the alternative recipe (Add i18n features): What do you do when you have a
defaultLocale
without a prefix like/en/
? (It's the default behaviour in the new feature) The answer is simple: you use/[...locale]
and handle theundefined
value ingetStaticPaths
. Pretty simple, but if an Astro beginner isn't aware of this possibility, the guide and the recipe appear to be missing critical info. Here's an example of what should be done:By itself, the guide about the Astro feature does not give enough examples to get you started. You have to use
/[...locale]
and adapt the following tips from the recipe Add i18n features to have a working website:useTranslations(Astro.currentLocale)
),The Astro feature is very flexible, so I don't see why people would want to not use it. That's why I feel like the recipe page is "legacy" and could be merged into the new guide for easier access to these clever tips.
🖥️ Reproduction in StackBlitz (if reporting incorrect content or code samples)
No response
The text was updated successfully, but these errors were encountered: