Skip to content

update ja docs #1182

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

Merged
merged 25 commits into from
Mar 8, 2018
Merged

update ja docs #1182

merged 25 commits into from
Mar 8, 2018

Conversation

kazupon
Copy link
Member

@kazupon kazupon commented Mar 8, 2018

No description provided.

@kazupon
Copy link
Member Author

kazupon commented Mar 8, 2018

/ping @re-fort @potato4d @lmiller1990 @38elements
どなたかレビューしてもらえると助かります! 🙏

@@ -1,4 +1,4 @@
# スコープ付き CSS
**再帰されたコンポーネントの子孫セレクタには気をつけてください!** セレクタ `.a .b` を持つ CSS ルールの場合、`.a` にマッチする要素に再帰的な子コンポーネントが含まれている場合、その子コンポーネントのすべての `.b` はルールにマッチします。# スコープ付き CSS

Copy link

@38elements 38elements Mar 8, 2018

Choose a reason for hiding this comment

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

# スコープ付き CSSの位置を1行目にする必要があると思います。


あるいは、 `vue-loader` の `postcss` オプションを使用して `*.vue` ファイル用の PostCSS の設定を指定することが出来ます
`vue-laoder` は内部的にはスタイルで PostCSS を扱うため、スタンドアロンの CSS ファイルに `postcss-loader` を適用する必要あります。プロジェクトに PostCSS 設定ファイルがある場合は、style ブロックに `lang="postcss"` を指定する必要はありません

Choose a reason for hiding this comment

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

vue-laoderをvue-loaderに変更する必要があると思います。

Copy link

@potato4d potato4d left a comment

Choose a reason for hiding this comment

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

既に指摘されているポイント以外、基本的に問題ないかと思います!
以下提案です。コメント内容について方針の決定、 vue-laoder のところの修正だけいただければ Approve で良さそうです。

@@ -80,7 +86,7 @@ module.exports = {

### postcss

> メモ: 11.0.0 以降では代わりに PostCSS の設定ファイル推奨されています。[使用法は `postcss-loader` と同じです](https://github.com/postcss/postcss-loader#usage)。
> メモ: 代わりに、vue ファイルと通常の CSS が同じ設定を共有することができるため、PostCSS 設定ファイルの使用を推奨されています。[使用法は `postcss-loader` と同じです](https://github.com/postcss/postcss-loader#usage)。
Copy link

Choose a reason for hiding this comment

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

他の場所ではシチュエーションに合わせて *.vue ファイル、もしくは Vue ファイルの記法のどちらかで統一されているので、統一したほうが良いかと思います。

Copy link
Member Author

Choose a reason for hiding this comment

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

確かにそうですね。

こちら、原文の方も、このようになっていたので、後でPR送りますね。

### esModule

> このオプションは v14.0 では削除されました。v14.0 以降では、`*.vue` ファイルは常に ES モジュールをエクスポーズします。
Copy link

Choose a reason for hiding this comment

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

は 単純に「エクスポート」としたほうが良いかもと思っていますがいかがでしょうか(� 「exports」 であることを明示したければこのままでも良いと思います!)

@kazupon
Copy link
Member Author

kazupon commented Mar 8, 2018

@38elements @potato4d
レビュー 👀 ありがとうございました!
指摘部分対応しましたので、確認お願いします! 🙏

Copy link

@potato4d potato4d left a comment

Choose a reason for hiding this comment

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

対応ありがとうございました!

@kazupon kazupon merged commit 6c1e53f into vuejs:master Mar 8, 2018
@kazupon kazupon deleted the docs/ja branch March 8, 2018 16:17
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.

3 participants