Skip to content
This repository has been archived by the owner on Dec 7, 2021. It is now read-only.

feat: Traditional Chinese language support #949

Merged
merged 19 commits into from
Apr 21, 2020

Conversation

aykhara
Copy link
Contributor

@aykhara aykhara commented Mar 24, 2020

Traditional Chinese translation verified by @rextangtw

Copy link
Contributor

@harvinchou harvinchou left a comment

Choose a reason for hiding this comment

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

The translation is good! Just provided some suggested changes to make them more align with Azure's official terms.

src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
rextangtw and others added 2 commits March 26, 2020 23:30
accepted suggestion. thanks for the feedback those make sense!

Co-Authored-By: Jason Chou <[email protected]>
Copy link
Contributor

@ShuyuanChang ShuyuanChang left a comment

Choose a reason for hiding this comment

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

I had suggested some general terms and pls check them.

src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
src/common/localization/zh-tw.ts Outdated Show resolved Hide resolved
thanks for the review, this is more close to normally a TW UI wording! appreciated!

Co-Authored-By: John Chang <[email protected]>
Copy link
Contributor

@mydiemho mydiemho left a comment

Choose a reason for hiding this comment

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

Please build a version locally and make sure all strings are rendered correctly. In the process of translation, a lot of the variables extrapolations are messed up. I mark some below but not all please double check

},
importProject: {
title: "匯入專案", // Import Project
confirmation: "您確定要將專案$ {project.file.name}的設定轉換為v2格式嗎?我們建議您首先備份專案文件。",
Copy link
Contributor

Choose a reason for hiding this comment

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

please remove the space between $ and {project.file.name}.

Copy link
Contributor

Choose a reason for hiding this comment

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

updated

// We recommend you backup the project file first.
},
messages: {
deleteSuccess: "已成功刪除$ {project.name}專案", // Successfully deleted ${project.name}
Copy link
Contributor

Choose a reason for hiding this comment

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

same

Copy link
Contributor

Choose a reason for hiding this comment

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

updated

},
addConnection: "新增連線", // Add Connection
messages: {
saveSuccess: "已成功保存$ {project.name}專案設定", // Successfully saved ${project.name} project settings
Copy link
Contributor

Choose a reason for hiding this comment

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

same

Copy link
Contributor

Choose a reason for hiding this comment

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

updated

// As for this VOTT tool, translate "Assets" to "Image data" in Traditional Chinese,
// as "Asset" can be confusing if directly translated.
totalAssetCount: "圖像數據總數", // Total Assets
visitedAssets: "已檢視的圖像數據($ {count})", // Visited Assets (${count})
Copy link
Contributor

Choose a reason for hiding this comment

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

same

Copy link
Contributor

Choose a reason for hiding this comment

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

updated

@sonarcloud
Copy link

sonarcloud bot commented Mar 27, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@rextangtw
Copy link
Contributor

Please build a version locally and make sure all strings are rendered correctly. In the process of translation, a lot of the variables extrapolations are messed up. I mark some below but not all please double check

thanks @mydiemho , I just updated 9 occurrence of those and committed.

@mydiemho mydiemho merged commit 0d5fdd0 into microsoft:develop Apr 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants