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

Fix format and typo #38

Open
wants to merge 2 commits into
base: feat/i18n-jp
Choose a base branch
from
Open

Fix format and typo #38

wants to merge 2 commits into from

Conversation

inductor
Copy link

@inductor inductor commented Mar 7, 2020

No description provided.

@inductor inductor requested review from gimKondo and hidesuke March 7, 2020 03:32
Copy link
Member

@gimKondo gimKondo left a comment

Choose a reason for hiding this comment

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

typo修正ありがとうございます。
文そのものの修正については疑問もあったので、Request changesでお返しします。

?後のスペースについては「全角スペースを空ける」「半角スペースを空ける」「スペースを空けない」が混在してますね。
まあ、このドキュメントの性質を考えると、何でもいいかな、と思ってます。

根拠は怪しいですが、主流なルールとしては全角スペースを空けると定めているのだとか。
https://liginc.co.jp/life/useful-info/127238

* 仕事のどのような面でもっと助けやコーチンがほしいですか
* 仕事上、もっと指示が欲しいところ、逆に指示されたくないところはどこでしょうか?
* 仕事のどのような面でもっと助けやコーチングがほしいですか
* 仕事上、もっと指示が欲しいところや、逆に指示されたくないところはどこでしょうか?
Copy link
Member

Choose a reason for hiding this comment

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

ここは元の文の方が良いかな、と思います。

* ここで自分の目標に向かってさらに進めることができますか? そうするためには何を変える必要がありますか?
* 仕事で学びがあると感じていますか? あなたが最近学んだ新しいことはなんですか? どんな分野を学びたいですか?
* ここで、あなたの大きな目標に向かって進んでいると感じますか? それはなぜでしょうか?
* ここで自分の目標に向かってさらに進めることはできますか?そうするためには何を変える必要がありますか?
Copy link
Member

Choose a reason for hiding this comment

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

この修正はどういった意図がありますか?

@hidesuke
Copy link
Member

hidesuke commented Mar 7, 2020

日本語文章作法としては「?」のあとは全角スペースですね。今回特に取り決めはありませんでしたが、個人的には全角でも半角でもスペースがあった方が読みやすいかなぁとは思います

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.

3 participants