Skip to content

Translate: CSS#3

Merged
nouvelle merged 4 commits intoGameWith:masterfrom
nouvelle:master
Feb 28, 2020
Merged

Translate: CSS#3
nouvelle merged 4 commits intoGameWith:masterfrom
nouvelle:master

Conversation

@nouvelle
Copy link

Fixes: #

🚨 Please review the guidelines for contributing and our code of conduct to this repository. 🚨
Please complete these steps and check these boxes (by putting an x inside the brackets) before filing your PR:

  • Check the commit's or even all commits' message styles matches our requested structure.
  • Check your code additions will fail neither code linting checks nor unit test.

Short description of what this resolves:

日本語への翻訳を行いました。

Proposed changes:

  • CSS

👍 Thank you!

@inosy22
Copy link
Member

inosy22 commented Feb 26, 2020

@nouvelle PRあげていただきありがとうございます!

またスタイルの話になり申し訳ないですが、
CSS という単語の前後には必ず半角スペースを含めてしまっていいかなと思っています。

JTFの標準スタイルには、半角と全角の混ざった単語の場合は半角スペースを入れないと定義されていますが、参考にしている Vue.js のドキュメントでは、半角英字の前後には一律で半角スペースを入れるようにしているようです。

↓ 参考までにカスタムスタイル定義で半角と全角の間のスペースを無視するルールを追加していることがこちらで確認できます。
#4

Copy link
Member

@inosy22 inosy22 left a comment

Choose a reason for hiding this comment

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

@nouvelle 上記以外、気になるところコメントいたしました!

@nouvelle
Copy link
Author

@inosy22 @noguchi-hiroshi

細かくチェックして頂きありがとうございます!
基本的にCSSの両脇には半角スペースを入れたのですが、
「クリティカルCSS」「リセットCSS」「ノーマライズCSS」のような1つの単語として成っているものは現状スペースは入れておりません。
これについてどう思われますでしょうか・・・?

これらも全てスペースあった方がよろしければ追って修正いたします!
お手数ですがご確認お願いいたします。

@53able
Copy link

53able commented Feb 26, 2020

@nouvelle コメントありがとうございます。
「クリティカルCSS」「リセットCSS」「ノーマライズCSS」も一様にアルファベット単語の前後は、スペースひとつ空けます。
理由は、「ひとつの単語」という認識で個人差のブレを回避するためです。
理系ドキュメントとしての洗練さは、フォーマットの統一だと考えているので、ご理解していただくと幸いです😉

@nouvelle
Copy link
Author

@53able

コメントありがとうございます😉
修正しましたのでご確認頂けると幸いです!

Copy link

@53able 53able left a comment

Choose a reason for hiding this comment

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

@nouvelle お疲れさまです!
いくつかの提案と、指摘をさせていただきました💦
何度も恐縮ですが、同意していただけましたら、修正よろしくお願いします👨‍💻

@nouvelle
Copy link
Author

@53able
細かく見てくださりありがとうございます!
修正しましたので、ご確認頂けると幸いです🙇‍♀️

Copy link

@53able 53able left a comment

Choose a reason for hiding this comment

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

@nouvelle
素晴らしいです💯
マージお願いします!

@nouvelle nouvelle merged commit 1e94eca into GameWith:master Feb 28, 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.

4 participants