Skip to content

Initialize adding the Simplified Chinese (zh-CN) translation of the fpm documents #12

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

Merged
merged 4 commits into from
Dec 1, 2021

Conversation

zoziha
Copy link
Collaborator

@zoziha zoziha commented Dec 1, 2021

  • add the Chinese translation.

CN Pages: https://zoziha.github.io/fpm-docs/cn/ (outdated)

@awvwgk
Copy link
Member

awvwgk commented Dec 1, 2021

I think cn is not matching any language sphinx-intl is aware of, however it has two entries for Chinese (see https://www.sphinx-doc.org/en/master/usage/configuration.html#options-for-internationalization):

  • zh_CN – Simplified Chinese
  • zh_TW – Traditional Chinese

Using either of those would also use translations for the blog and table of contents. If you prefer cn as path for the URL I can have a look how we can keep this path and still get the right translation from the sphinx-intl for the theme.

@awvwgk awvwgk added i18n Localisation and internationalisation of the documentation i18n: Chinese Relevant for Chinese translation, this thread might be in Chinese ;) labels Dec 1, 2021
@zoziha zoziha changed the title Initialize adding the Chinese translation of the fpm documents Initialize adding the Simplified Chinese (zh-CN) translation of the fpm documents Dec 1, 2021
@zoziha
Copy link
Collaborator Author

zoziha commented Dec 1, 2021

Thanks, I'll replace cn with zh_CN.

@zoziha zoziha marked this pull request as draft December 1, 2021 10:45
@zoziha zoziha force-pushed the add_Chinese_trans branch from 31cd846 to bb56781 Compare December 1, 2021 10:51
Language identifier: `zh_CN` -> `cn`.

Co-authored-by: Sebastian Ehlert <[email protected]>
Copy link
Member

@awvwgk awvwgk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot, I'll go ahead and merge.

@awvwgk awvwgk merged commit 1d40639 into fortran-lang:main Dec 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i18n: Chinese Relevant for Chinese translation, this thread might be in Chinese ;) i18n Localisation and internationalisation of the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants