Skip to content

docs: add Chinese Traditional (Taiwan)#930

Merged
coder3101 merged 16 commits intocpeditor:masterfrom
l10n-tw:l10n-tw
Jul 21, 2021
Merged

docs: add Chinese Traditional (Taiwan)#930
coder3101 merged 16 commits intocpeditor:masterfrom
l10n-tw:l10n-tw

Conversation

@toto6038
Copy link
Copy Markdown
Contributor

@toto6038 toto6038 commented Jul 14, 2021

Description

Translate README.md, CONTRIBUTING.md, DONATE.md into Chinese Traditional (Taiwan), and add the link to zh-TW in original README.md

Related Issues / Pull Requests

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate)

Checklist

  • If the key of a setting is changed, the old attribute is updated or it is resolved in SettingsUpdater.
  • If there are changes of the text displayed in the UI, they are wrapped in tr() or QCoreApplication::translate().
  • If needed, I have opened a pull request or an issue to update the documentation.
  • If these changes are notable, they are documented in CHANGELOG.md.

Additional text

@ouuan
Copy link
Copy Markdown
Member

ouuan commented Jul 15, 2021

If you are willing to translate the website, I can prepare the structure now so that we can link to it in this PR.

@toto6038
Copy link
Copy Markdown
Contributor Author

If you are willing to translate the website, I can prepare the structure now so that we can link to it in this PR.

I'm glad to do that.

@ouuan
Copy link
Copy Markdown
Member

ouuan commented Jul 15, 2021

I've opened cpeditor/cpeditor.github.io#198, and you need to accept the membership invitation of @cpeditor to push to that branch.

@neko-para
Copy link
Copy Markdown
Contributor

Will you translate the UI of the excutable? (just wondering)

@ouuan
Copy link
Copy Markdown
Member

ouuan commented Jul 15, 2021

Will you translate the UI of the excutable? (just wondering)

I think the UI translation could be in another PR because it doesn't affect the translation of the documentation. However, with the translation of the website, we should link to translated pages in the docs.

@coder3101
Copy link
Copy Markdown
Member

coder3101 commented Jul 15, 2021

Will you translate the UI of the excutable? (just wondering)

Yes. I think before translating the website, the application should be translated, it is going the other way around.

Ideal ordering should be

  1. Translate Application
  2. Translate README/CONTRIBUTE etc
  3. Translate the website

@toto6038
Copy link
Copy Markdown
Contributor Author

toto6038 commented Jul 15, 2021

Comparing to website or introduction, I'm less confident in translating UI strings. However I can still try my best to do it.

@toto6038
Copy link
Copy Markdown
Contributor Author

I currently proceed to do UI localization. Do I have to duplicate and modify the existing file in translation directory, or is there a way to get the English ts file?

@ouuan
Copy link
Copy Markdown
Member

ouuan commented Jul 17, 2021

I currently proceed to do UI localization. Do I have to duplicate and modify the existing file in translation directory, or is there a way to get the English ts file?

You can use the zh_TW-app branch. And you need to accept the membership invitation in your email inbox or at https://github.com/cpeditor, otherwise, you won't have permission to push to that branch.

@ouuan
Copy link
Copy Markdown
Member

ouuan commented Jul 17, 2021

BTW, do you know anyone that could help review these translations? It will be better if another competitive programmer from Taiwan could review these translations.

@toto6038
Copy link
Copy Markdown
Contributor Author

Thanks for what you've done. I can find someone to review the translation.

Copy link
Copy Markdown
Contributor Author

@toto6038 toto6038 left a comment

Choose a reason for hiding this comment

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

finished review

@coder3101
Copy link
Copy Markdown
Member

You should update the PR by merging master branch to it. Readme was updated to include snap version in release table.

@toto6038
Copy link
Copy Markdown
Contributor Author

PR updated

neko-para
neko-para previously approved these changes Jul 20, 2021
coder3101
coder3101 previously approved these changes Jul 20, 2021
Copy link
Copy Markdown
Member

@ouuan ouuan left a comment

Choose a reason for hiding this comment

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

Link to the zh_tw version of cpeditor.org

@toto6038 toto6038 dismissed stale reviews from coder3101 and neko-para via c60bec0 July 20, 2021 05:30
@toto6038 toto6038 requested a review from ouuan July 20, 2021 05:32
@ouuan
Copy link
Copy Markdown
Member

ouuan commented Jul 20, 2021

Do you want to work on cpeditor/cpeditor.github.io#198 now or later? I can merge it now if you want to create a new PR later.

@toto6038
Copy link
Copy Markdown
Contributor Author

The translation of website will be worked on later.

@swiftqwq
Copy link
Copy Markdown
Member

@toto6038 please update the files part in .all-contributorsrc.

@toto6038
Copy link
Copy Markdown
Contributor Author

.all-contributorsrc: file edited

@ouuan
Copy link
Copy Markdown
Member

ouuan commented Jul 20, 2021

dist/linux/cpeditor.desktop and cmake/cpeditor.appdata.xml.in

Copy link
Copy Markdown
Member

@ouuan ouuan left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@coder3101 coder3101 merged commit 82ae1ca into cpeditor:master Jul 21, 2021
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.

5 participants