Skip to content
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

fixes #330 javascript/ajax/index.md 和訳の見直し #331

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

stein2nd
Copy link

@stein2nd stein2nd commented Nov 6, 2024

Fixes #330

和訳 85 行目:コールバックのセクション
=> コールバックのセクション

原文 216 行目:PHP section
和訳 218 行目:PHP のセクション
…原文の時点で、リンク先が間違ってる様な気がします(セクションが自分自身を指してることになるのでは?)。
且つ、「リンクテキスト」と「タイトル」が甚だしく乖離してますし。

textlint の再実施
和訳 11 行目:ブラウザ => ブラウザー
和訳 21 行目:ブラウザ => ブラウザー
和訳 211 行目:ブラウザ => ブラウザー

ここで、[jQuery の記事のスニペット](https://developer.wordpress.org/plugins/javascript/jquery/#selector-and-event)にある「do stuff」部分を定義します。ここでは、[`$.post()`](https://api.jquery.com/jQuery.post/ "jQuery リファレンス") メソッドを使用し、そのパラメータは、POST リクエストを送信する URL、送信するデータ、そしてサーバーのレスポンスを処理するコールバック関数、の3つになります。でも、その前に、ちょっとした下準備があります。後でコールバック関数で使うために、次のように割り当てします。目的は[コールバックのセクション](https://developer.wordpress.org/plugins/javascript/ajax/#callback "Page section")で詳しく説明します。
ここで、[jQuery の記事のスニペット](https://developer.wordpress.org/plugins/javascript/jquery/#selector-and-event)にある「do stuff」部分を定義します。ここでは、[`$.post()`](https://api.jquery.com/jQuery.post/ "jQuery リファレンス") メソッドを使用し、そのパラメータは、POST リクエストを送信する URL、送信するデータ、そしてサーバーのレスポンスを処理するコールバック関数、の3つになります。でも、その前に、ちょっとした下準備があります。後でコールバック関数で使うために、次のように割り当てします。目的は[コールバックのセクション](https://developer.wordpress.org/plugins/javascript/ajax/#callback "コールバック")で詳しく説明します。
Copy link
Member

Choose a reason for hiding this comment

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

こちら、原文リポジトリにもPRを提出していただくことは可能でしょうか。

Copy link
Author

Choose a reason for hiding this comment

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

明日以降にでも、原文側に PR を出してみようと思います。

Copy link
Member

Choose a reason for hiding this comment

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

原文側にもPRを立てましたら、その PR 番号をここにリンクしていただけると助かります。

@stein2nd
Copy link
Author

stein2nd commented Nov 9, 2024

下記の様に、issueを立てました。
WordPress#40

内容は、↓こんな感じです。

  • javascript/ajax/index.md ファイルの111行目のURLリンクの「表示テキスト」と「タイトル」が、著しく乖離してるよ。
  • 「リンク先」は、自分自身の様だけど、妥当? 例えば、javascript/enqueuing/index.mdにリンクするのが妥当じゃね?

@stein2nd
Copy link
Author

原文側のfrantorres氏のコメントによれば、フレーズ自体の変更が必要では?との事です。

@t-hamano
Copy link
Member

ありがとうございます!
原文側にPRを提出していただくことは可能でしょうか?

  • 原文側にPR提出
  • PRがマージされる
  • 日本語リポジトリをUpstreamに追従させる
  • このPRを更新する

の順で進められたらなと思います。

@stein2nd
Copy link
Author

stein2nd commented Nov 28, 2024

先程、原文側に修正PRを提出しました。
WordPress#43

@stein2nd
Copy link
Author

stein2nd commented Dec 4, 2024

こちら、 frantorres さんのレビュー待ちです。

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.

javascript/ajax/index.md 和訳の見直し
2 participants