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

docs(basic/overview)を翻訳 #33

Merged
merged 4 commits into from Sep 11, 2016
Merged

docs(basic/overview)を翻訳 #33

merged 4 commits into from Sep 11, 2016

Conversation

ghost
Copy link

@ghost ghost commented Sep 8, 2016

翻訳のチェックをお願いします。

この章は翻訳が完了しているか?

はい(pr_state: PLEASE_REVIEW)

このPRには何が含まれますか?

翻訳

それ以外に伝えておきたいこと

  • 動作例 と書いて、リンクが張られたまま日本語訳を表示できたのでそうしている。
  • {_liveLink} -> 動作例へのリンクとした。

  • 訳語では表の見出しのレイアウトが崩れるため、<b style="white-space: nowrap">クイックスタート</b>と、一箇所だけインラインスタイルシートを挿れた。Slackでは英語のままにするのもよいとの議論があったが、訳文の中で見出しを指し示すことやナビゲーションリンクにおける訳とも合わせるために、ここでも日本語に訳したため。
    インラインスタイルシートはこの1箇所にしか適用していないため、将来のUpstreamの変更への対応の負担は軽いと思われる。

@lacolaco lacolaco mentioned this pull request Sep 10, 2016
10 tasks
ntaoo added 2 commits September 10, 2016 11:42
# 残作業

* 見出しの翻訳
* 時間をあけての全体の校正
- tdの行を修正した。
- <live-example name="architecture">動作例</live-example> と書いて、リンクが張られたまま日本語訳を表示できたのでそうしている。
- #{_liveLink} -> 動作例へのリンクとした。
@ghost ghost changed the title [WIP]basic/overview翻訳中 docs(basic/overview)を翻訳 Sep 10, 2016
@lacolaco lacolaco added this to the Be Ready milestone Sep 10, 2016

Use the [angular.io Github repo](https://github.com/angular/angular.io) for **documentation** issues and pull requests.

**ドキュメンテーション**の課題とプルリクエストについては、
[angular.ioのGithubリポジトリ](https://github.com/angular/angular.io)を使用してください。
Copy link
Author

Choose a reason for hiding this comment

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

angular-jaのリンクのほうが良かったかもしれない

Copy link
Collaborator

Choose a reason for hiding this comment

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

原文では"Github"となっていますが正式なブランドは"GitHub"なので日本語では修正しておきましょう

Copy link
Collaborator

@lacolaco lacolaco Sep 10, 2016

Choose a reason for hiding this comment

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

angular-cnではangular.ioのままなのでとりあえず踏襲しておきましょう。

@lacolaco
Copy link
Collaborator

@ntaoo レビューしました。コメントを残しているので対応をお願いします。

@lacolaco lacolaco merged commit 5438280 into angular:master Sep 11, 2016
@lacolaco
Copy link
Collaborator

Thanks!

@ghost
Copy link
Author

ghost commented Sep 11, 2016

👍

@ghost ghost deleted the translation/basic/overview branch September 11, 2016 00:24
slm-sum pushed a commit to slm-sum/angular-ja that referenced this pull request Sep 17, 2016
lacolaco pushed a commit that referenced this pull request Jan 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant