Skip to content
This repository has been archived by the owner on Aug 3, 2024. It is now read-only.

Preliminary i18n support (non-user content) #298

Closed
wants to merge 14 commits into from
Closed

Preliminary i18n support (non-user content) #298

wants to merge 14 commits into from

Conversation

triphora
Copy link
Contributor

@triphora triphora commented Aug 5, 2021

Even though this is currently being debated in Discord, I'm still going to make this PR just to show that it is possible. Even if it ends up being rejected /shrug

Inspired by HangarMC

To-do

  • Implement i18n
  • Auto-detect language from browser settings
  • Move everything from the pages to the 'dictionary' (done for everything but legal and about)
  • Translatable head titles & field placeholders
  • Add a switcher for language (probably next to theme switcher)
  • Add Crowdin support?
  • Crowdin is optimal because we can receive a open source license from them.

If someone wants to help out with this, feel free to PR to my repo (i18n branch)
Vercel deploy here, but Vercel doesn't seem to like it (it works just fine locally)

closes modrinth/translations#11

Inspired by [HangarMC](https://github.com/HangarMC/Hangar/tree/master/frontend)

To-Do
[x] Implement i18n
[x] Auto-detect language from browser settings
[ ] Move everything from the pages to the 'dictionary' (done for index)
[ ] Add a switcher for language (probably next to theme switcher)
[ ] Add Crowdin support?
 - Crowdin is optimal because we can receive a open source license from them.
The legend lives on from the Chippewa on down to the big lake they call Gitchee Gumee
@triphora
Copy link
Contributor Author

Going to close this because I feel as if this is far more of a long-term thing that is a ways off. Not going to delete the branch, though, so I'll probably pick this up again some time in the future.

@triphora triphora closed this Nov 29, 2021
@triphora triphora deleted the i18n branch March 1, 2022 23:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant