Skip to content
This repository was archived by the owner on Jan 20, 2025. It is now read-only.

Conversation

@hirotaka
Copy link
Contributor

@hirotaka hirotaka commented Jan 16, 2020

概要

tutorial/part-two の翻訳になります。

チェックリスト

  • 翻訳スタイルガイド に目を通しました。
  • Translation Guide に目を通しました。
  • textlint を使って校正を行いました(推奨)
  • Allow edits from maintainers にチェックを入れました。

しばらく待ってもレビューが終わらなかったり、必要なレビュー数が足りない状態が続いた場合は、こちらからメンテナーを探して、@を付けてメンションを飛ばしてください。

@uetchy uetchy added the status: awaiting review A pull request that is currently awaiting a reviewer's response label Jan 16, 2020
@hirotaka hirotaka requested a review from smorimoto as a code owner January 17, 2020 03:47
@hirotaka hirotaka requested a review from Naturalclar January 17, 2020 03:47
Copy link
Contributor

@uetchy uetchy left a comment

Choose a reason for hiding this comment

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

👍

```

> Note: The placement of the example css file in a `/src/styles/` folder is arbitrary.
> ヒント: 例の css ファイルを `/src/styles/` フォルダーに配置するのは任意です。
Copy link
Contributor

Choose a reason for hiding this comment

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

「例のcssファイル〜」で「以前CSSファイルについて言及したのかな...?」と引っかかりました。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

たしかにひっかかりますね。

#### CSS-in-JS に関するお勧めの読み物

If you're interested in further reading, check out [Christopher "vjeux" Chedeau's 2014 presentation that sparked this movement](https://speakerdeck.com/vjeux/react-css-in-js) as well as [Mark Dalgleish's more recent post "A Unified Styling Language"](https://medium.com/seek-blog/a-unified-styling-language-d0c208de2660).
さらに詳しものを読んでみたい場合は、[このムーブメントを引き起こした Christopher "vjeux" Chedeau2014 年のプレゼンテーション](https://speakerdeck.com/vjeux/react-css-in-js)[Mark Dalgleish の最近の投稿"A Unified Styling Language"](https://medium.com/seek-blog/a-unified-styling-language-d0c208de2660)を参考にしてください。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
さらに詳しものを読んでみたい場合は[このムーブメントを引き起こした Christopher "vjeux" Chedeau の 2014 年のプレゼンテーション](https://speakerdeck.com/vjeux/react-css-in-js)[Mark Dalgleish の最近の投稿"A Unified Styling Language"](https://medium.com/seek-blog/a-unified-styling-language-d0c208de2660)を参考にしてください。
さらに詳しく読んでみたい場合は[このムーブメントを引き起こした Christopher "vjeux" Chedeau の 2014 年のプレゼンテーション](https://speakerdeck.com/vjeux/react-css-in-js)[Mark Dalgleish の最近の投稿"A Unified Styling Language"](https://medium.com/seek-blog/a-unified-styling-language-d0c208de2660)を参考にしてください。

> ヒント: 例の css ファイルを `/src/styles/` フォルダーに配置するのは任意です。
#### Include the stylesheet in `gatsby-browser.js`
#### `gatsby-browser.js` にスタイルシートをインクルード
Copy link
Contributor

Choose a reason for hiding this comment

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

ここの制御文字が気になりました。

Copy link
Contributor Author

@hirotaka hirotaka Jan 19, 2020

Choose a reason for hiding this comment

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

なんだろう🤔
削除しました。

> ヒント: 例の css ファイルを `/src/styles/` フォルダーに配置するのは任意です。
#### Include the stylesheet in `gatsby-browser.js`
#### `gatsby-browser.js` にスタイルシートをインクルード
Copy link
Contributor

Choose a reason for hiding this comment

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

インクルードは日本語にしても良いと思います。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

了解です!

@uetchy uetchy added the status: awaiting author response A pull request that is awaiting author's response label Jan 17, 2020
@hirotaka hirotaka requested a review from uetchy January 19, 2020 03:51
@hirotaka hirotaka requested a review from smorimoto January 19, 2020 04:19
@hirotaka hirotaka removed the status: awaiting author response A pull request that is awaiting author's response label Jan 19, 2020
---

<!-- Idea: Create a glossary to refer to. A lot of these terms get jumbled -->
<!-- アイデア: 参照する用語集を作成します。これらの用語の多くは混乱します -->
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<!-- アイデア: 参照する用語集を作成します。これらの用語の多くは混乱します -->
<!-- アイデア: 参照できる用語集を作成するのはどうでしょう。これらの用語の多くは混乱を招いています。 -->

## このチュートリアルの内容は?

In this part, you're going to explore options for styling Gatsby websites and dive deeper into using React components for building sites.
このパートでは、Gatsby Web サイトのスタイリングのオプションを詳しくみていき、React コンポーネントを使用してサイトを構築する方法について詳しく説明していきます。
Copy link
Contributor

Choose a reason for hiding this comment

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

詳しくが連続しているのが引っかかりました。

## グローバルスタイルの使用

Every site has some sort of global style. This includes things like the site's typography and background colors. These styles set the overall feel of the site — much like the color and texture of a wall sets the overall feel of a room.
すべてのサイトには、ある種のグローバルなスタイルがあります。これには、サイトのタイポグラフィや背景色などを含みます。これらのスタイルは、サイト全体の雰囲気を設定します。壁の色や質感が部屋の全体的な雰囲気を設定するのに似ています。
Copy link
Contributor

Choose a reason for hiding this comment

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

設定より決定の方が自然ではないですか?

```

You now have a new Gatsby site (based on the Gatsby "hello world" starter) with the following structure:
次の構成の新しい Gatsby サイト(Gatsby の「hello world」スターターに基づいた)が作成されました。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
次の構成の新しい Gatsby サイト(Gatsby の「hello world」スターターに基づいた)が作成されました。
Gatsby の「hello world」スターターに基づいた新しい Gatsby サイトが作成されました:

ファイル名が通常の `.css` ではなく `.module.css` で終わっていることに気付いたことでしょう。これは、この CSS ファイルをプレーンな CSS ではなく CSS module として処理する必要があることを Gatsby に伝えます。

3. Create a new page component by creating a file at
3. 次のファイルに新しいページコンポーネント作成します。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
3. 次のファイルに新しいページコンポーネント作成します
3. 次のファイルに新しいページコンポーネントを作成します

#### ✋ CSS Modules を使用してコンポーネントのスタイルを設定

In this section, you'll create a list of people with names, avatars, and short Latin biographies. You'll create a `<User />` component and style that component using a CSS module.
このセクションでは、名前、アバターと短いラテン語の経歴(Lorem Ipsum - ダミーテキスト)を含む人々のリストを作成します。`<User />` コンポーネントと CSS モジュールを使用したスタイルを作成します。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
このセクションでは、名前、アバターと短いラテン語の経歴(Lorem Ipsum - ダミーテキスト)を含む人々のリストを作成します。`<User />` コンポーネントと CSS モジュールを使用したスタイルを作成します。
このセクションでは、名前、アバターと短いラテン語のプロフィール(=Lorem Ipsum - ダミーテキストのこと)を含む人々のリストを作成します。`<User />` コンポーネントと CSS モジュールを使用したスタイルを作成します。

1. `src/pages/about-css-modules.module.css` という CSS のファイルを作成します。

2. Paste the following into the new file:
2. 新しいファイルに次を貼り付けます。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
2. 新しいファイルに次を貼り付けます
2. 新しいファイルに次のコードを貼り付けます

このセクションでは、名前、アバターと短いラテン語の経歴(Lorem Ipsum - ダミーテキスト)を含む人々のリストを作成します。`<User />` コンポーネントと CSS モジュールを使用したスタイルを作成します。

1. Create the file for the CSS at `src/pages/about-css-modules.module.css`.
1. `src/pages/about-css-modules.module.css` という CSS のファイルを作成します。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
1. `src/pages/about-css-modules.module.css` という CSS のファイルを作成します
1. `src/pages/about-css-modules.module.css` という名前で CSS ファイルを作成します

#### CSS-in-JS に関するお勧めの読み物

If you're interested in further reading, check out [Christopher "vjeux" Chedeau's 2014 presentation that sparked this movement](https://speakerdeck.com/vjeux/react-css-in-js) as well as [Mark Dalgleish's more recent post "A Unified Styling Language"](https://medium.com/seek-blog/a-unified-styling-language-d0c208de2660).
さらに詳しものを読んでみたい場合は、[このムーブメントを引き起こした Christopher "vjeux" Chedeau2014 年のプレゼンテーション](https://speakerdeck.com/vjeux/react-css-in-js)[Mark Dalgleish の最近の投稿「A Unified Styling Language](https://medium.com/seek-blog/a-unified-styling-language-d0c208de2660)を参考にしてください。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
さらに詳しものを読んでみたい場合は[このムーブメントを引き起こした Christopher "vjeux" Chedeau の 2014 年のプレゼンテーション](https://speakerdeck.com/vjeux/react-css-in-js)[Mark Dalgleish の最近の投稿「A Unified Styling Language」](https://medium.com/seek-blog/a-unified-styling-language-d0c208de2660)を参考にしてください。
さらに詳しく読んでみたい場合は[このムーブメントを引き起こした Christopher "vjeux" Chedeau の 2014 年のプレゼンテーション](https://speakerdeck.com/vjeux/react-css-in-js)[Mark Dalgleish の最近の投稿「A Unified Styling Language」](https://medium.com/seek-blog/a-unified-styling-language-d0c208de2660)を参考にしてください。

@hirotaka hirotaka added the status: WIP Work in progress label Jan 20, 2020
@hirotaka hirotaka removed the status: WIP Work in progress label Jan 22, 2020
@hirotaka
Copy link
Contributor Author

全体的に見直したので再度レビューお願いいたします 🙇

@uetchy uetchy added status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned and removed status: awaiting review A pull request that is currently awaiting a reviewer's response labels Jan 22, 2020
Copy link
Contributor

@Naturalclar Naturalclar left a comment

Choose a reason for hiding this comment

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

👍

@Naturalclar Naturalclar merged commit 8281dc3 into master Jan 22, 2020
@Naturalclar Naturalclar deleted the tutorial/part-two branch January 22, 2020 14:07
@uetchy uetchy removed the status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned label Jan 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants