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

Conversation

@u00000ka
Copy link
Contributor

@u00000ka u00000ka commented Jan 25, 2020

概要

docs/babel を翻訳しました。レビューよろしくお願いします。

チェックリスト

  • 翻訳スタイルガイド に目を通しました。
  • Translation Guide に目を通しました。
  • textlint を使って校正を行いました。
  • 文章全体を最初から読み直して不自然な箇所が無いことを確認しました。
  • Allow edits from maintainers にチェックを入れました。

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

補足

Ref: #1

@smorimoto smorimoto added the status: awaiting review A pull request that is currently awaiting a reviewer's response label Jan 25, 2020
Comment on lines 5 to 6
Gatsby [Babel](https://babeljs.io/) というすばらしいプロジェクトを使っており、
モダンな JavaScript をサポートしていない古いブラウザーへのサポートをしつつ、モダンな JavaScript の記述が可能です。
Copy link
Contributor

Choose a reason for hiding this comment

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

ここは1行で以下にように書いてしまうのが良いと思います。

Gatsby は、[Babel](https://babeljs.io/) という素晴らしいプロジェクトを使い、古いブラウザをサポートしつつ、モダンなJavaScriptを書けるようにしています。

Comment on lines 10 to 11
Gatsby は基本的に、メジャーなブラウザーの最新 2 つのバージョン、 IE 9 以上 、
そして 1% 以上のシェアがあればどのようなブラウザーもサポートしています。
Copy link
Contributor

Choose a reason for hiding this comment

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

ここも同様に1行で、そして以下にように書けると思います。

Gatsby はデフォルトでメジャーなブラウザの最新の 2 つのバージョンと IE9 以上、そして 1% 以上のシェアのあるブラウザをサポートしています。

Polyfills are also automatically added — no more shipping code which mysteriously
breaks on older browsers!
つまり JavaScript は自動的にコンパイルされ、古いブラウザー上でも動作が保証されているということです。
Polyfills も自動的に追加されますので、古いブラウザー上で謎の動きをするコードはもうありません!
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
Polyfills も自動的に追加されますので、古いブラウザー上で謎の動きをするコードはもうありません
ポリフィルも自動的に追加されるため、コードが古いブラウザー上で不可解に壊れることはもうありません

Support](/docs/browser-support/) docs page for how to instruct Gatsby on which
browsers you support and then Babel will start compiling for only these
browsers.
もし最新のブラウザーのみの対応で良いなら、[サポートするブラウザー](/docs/browser-support/)のページをご覧ください。
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
もし最新のブラウザーのみの対応で良いなら[サポートするブラウザー](/docs/browser-support/)のページをご覧ください。
もし、新しいブラウザのみの対応で良いなら[サポートするブラウザー](/docs/browser-support/)のページをご覧ください。

どのように Gatsby を設定すれば 、あなたがサポートしたいブラウザーのみを Babel がコンパイルしてくれるのか記載しています。

## How to use a custom .babelrc file
## .babelrc ファイルをどのようにカスタムして使うか
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
## .babelrc ファイルをどのようにカスタムして使うか
## どのようにカスタム .babelrc ファイルを使うのか

Comment on lines 48 to 49
より高度な設定をしたければ、 [`babel-preset-gatsby`](https://github.com/gatsbyjs/gatsby/tree/master/packages/babel-preset-gatsby) からデフォルト値をコピーし
必要に応じてカスタマイズするのも良いでしょう。
Copy link
Contributor

Choose a reason for hiding this comment

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

不要なスペースを消し、他と同様に1行にしてください。

より高度な設定をしたければ、[`babel-preset-gatsby`](https://github.com/gatsbyjs/gatsby/tree/master/packages/babel-preset-gatsby) からデフォルト値をコピーし、必要に応じてカスタマイズすることも出来ます。

@smorimoto
Copy link
Contributor

英語の場合、文の途中で改行しても単語間に半角スペースが入るだけなので問題ないのですが、日本語の場合文の途中で改行すると、スタイルが崩れるので途中改行は避けてください。

@uetchy uetchy added status: awaiting author response A pull request that is awaiting author's response and removed status: awaiting review A pull request that is currently awaiting a reviewer's response labels Jan 27, 2020
@u00000ka
Copy link
Contributor Author

レビューありがとうございます。日本語での改行の件承知しました。
修正したのでご確認よろしくお願いします。

and add additional plugins, presets, and pass options to `babel-preset-gatsby`, e.g. `targets`.
Gatsby には、ほとんどのサイトで動くようなデフォルトの .babelrc ファイルがセットアップされています。
もしあなたが Babel のプリセットやプラグインをカスタムしたい場合、ご自身専用の `.babelrc` をサイトのルートに作成してください。
そして `babel-preset-gatsby`](https://github.com/gatsbyjs/gatsby/tree/master/packages/babel-preset-gatsby) をインポートし、
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
そして `babel-preset-gatsby`](https://github.com/gatsbyjs/gatsby/tree/master/packages/babel-preset-gatsby) をインポートし、
そして [`babel-preset-gatsby`](https://github.com/gatsbyjs/gatsby/tree/master/packages/babel-preset-gatsby) をインポートし、

@u00000ka
Copy link
Contributor Author

u00000ka commented Feb 1, 2020

レビューありがとうございます。記号が欠けていたのを修正しました。
ご確認よろしくお願いします。

@uetchy uetchy added status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned and removed status: awaiting author response A pull request that is awaiting author's response labels Feb 1, 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.

LGTM! ありがとうございます!

@Naturalclar Naturalclar merged commit 2ac1f26 into gatsbyjs:master Feb 1, 2020
@uetchy uetchy removed the status: awaiting more approval A pull request that is awaiting reviewer #2 to be assigned label Feb 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants