Skip to content
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

Sorting the display order of languages #453

Merged
merged 2 commits into from
Jul 5, 2024

Conversation

Okabe-Junya
Copy link
Contributor

Background

On the TAG ENV website, there was no specific order for displaying languages, and the notations were inconsistent. As shown in the image, Chinese was written as "中文" (in the local language), while Korean was written as "Korean" in English.

Screenshot 2024-07-03 at 22 58 19

Change List

  • Change all languages to Local Language (English notation)
  • Sort the weight to be in alphabetical order based on the English notation

This is the same method used by CNCF/Glossary. Please note that CNCF/Glossary is one of the most advanced projects regarding localization

Signed-off-by: Junya Okabe <okabe.junya.qj@alumni.tsukuba.ac.jp>
Signed-off-by: Junya Okabe <okabe.junya.qj@alumni.tsukuba.ac.jp>
Copy link

netlify bot commented Jul 3, 2024

Deploy Preview for tag-env-sustainability ready!

Name Link
🔨 Latest commit 71f189c
🔍 Latest deploy log https://app.netlify.com/sites/tag-env-sustainability/deploys/66855a2a9d59570008787bbf
😎 Deploy Preview https://deploy-preview-453--tag-env-sustainability.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@leonardpahlke leonardpahlke left a comment

Choose a reason for hiding this comment

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

Thank you so much!

@leonardpahlke leonardpahlke merged commit e29ad20 into cncf:main Jul 5, 2024
6 checks passed
@Okabe-Junya Okabe-Junya deleted the sort-languages branch July 5, 2024 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants