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

Conversation

@harapeko
Copy link
Contributor

@harapeko harapeko commented Feb 13, 2020

概要

https://www.gatsbyjs.org/docs/css-libraries-and-frameworks の翻訳です

チェックリスト

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

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

補足

  1. 原文にはないリンクを追加しました。詳細はPR内コメントをご確認ください translate docs/css-libraries-and-frameworks #175 (review)

Ref: #1

Gatsby プロジェクトでは多くの CSS スタイルライブラリやフレームワークを使用できます。

These libraries are not full-on approaches to styling, and generally work no matter which styling approach you've chosen for your website. They require installing third-party libraries, often with the help of Gatsby community plugins.
これらのライブラリは[スタイリング手法](/docs/styling/)には関係しません。また、一般的にはどのスタイリング手法をウェブサイトに選択したとしても機能します。
Copy link
Contributor Author

@harapeko harapeko Feb 13, 2020

Choose a reason for hiding this comment

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

how?
スタイリング手法(styling approach) にリンクを設けました。

why?
このリンクがあることで スタイリング手法(styling approach) が本ページ及び、GuideListのことを指しているのではなく、リンク先の大別3種の事を指していることを明らかにできると思ったからです。

逆にこのリンクがないと本ページのことを指していると誤解を招く可能性があり、必要と判断しました。

Copy link
Contributor

Choose a reason for hiding this comment

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

@harapeko 提案ありがとうございます!良い変更だと思います。
本リポジトリはあくまで原文の翻訳という立ち位置なので、 まずは Gatsby 本家のドキュメントに変更を提案できると良いと思います:bow:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

提案してきました!gatsbyjs/gatsby#21491

本家Contributeの導きをありがとうございます👍

@harapeko harapeko self-assigned this Feb 13, 2020
@smorimoto smorimoto added the status: awaiting review A pull request that is currently awaiting a reviewer's response label Feb 14, 2020
These libraries are not full-on approaches to styling, and generally work no matter which styling approach you've chosen for your website. They require installing third-party libraries, often with the help of Gatsby community plugins.
これらのライブラリは[スタイリング手法](/docs/styling/)には関係しません。また、一般的にはどのスタイリング手法をウェブサイトに選択したとしても機能します。

大抵の場合は Gatsby コミュニティプラグインの助けを借りてサードパーティライブラリをインストールする必要があります。
Copy link
Contributor

Choose a reason for hiding this comment

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

改行を原文と合わせていただけると助かります:bow:
原文に変更があったときにDiffを見やすくするためです。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

🙏

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.

一つSuggestしたので確認お願いします:pray:

uetchy
uetchy previously approved these changes Feb 15, 2020
@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 Feb 15, 2020
why?
Easy diff with original.

branch
docs/css-libraries-and-frameworks
@harapeko
Copy link
Contributor Author

@Naturalclar Suggestionを取り込みました!

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.

ありがとうございます!:+1:

@Naturalclar Naturalclar 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 15, 2020
@smorimoto smorimoto merged commit a064a85 into gatsbyjs:master Feb 16, 2020
@smorimoto
Copy link
Contributor

Thanks @harapeko for your contribution!

@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