i18n(ja): update reference file plugins.md#3709
i18n(ja): update reference file plugins.md#3709Yusaku01 wants to merge 3 commits intowithastro:mainfrom
Conversation
|
✅ Deploy Preview for astro-starlight ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
github-actions
bot
added
i18n
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
morinokami
left a comment
morinokami
left a comment
There was a problem hiding this comment.
対応ありがとうございます、何点かコメントしましたのでご確認をお願いします！
ただ、この plugins と 18n の新規内容が互いにリンクし合っているようなため、このままではマージできない気がします。なので i18n 側の修正にこの変更を合流させる必要がありそうです。
| 既存の設定オプションをオーバーライドせず拡張するには、既存の値を新しい値へと展開します。以下の例では、`config.social`を新しい`social`オブジェクトに展開し、既存の設定に新しい[`social`](/ja/reference/configuration/#social)メディアアカウントを追加しています。 | ||
| 既存の設定オプションをオーバーライドせず拡張するには、既存の値を新しい値へと展開します。以下の例では、`config.social`を新しい`social`配列に展開し、既存の設定に新しい[`social`](/ja/reference/configuration/#social)アカウントを追加しています。 |
There was a problem hiding this comment.
二つ目の文について質問なのですが、原文は
In the following example, a new social media account is added to the existing configuration by spreading config.social into the new social array
となっており、「a new social media account」なので「メディア」という語は削除する必要もないように思ったのですが、これは意図的な変更でしょうか？
There was a problem hiding this comment.
すみません、こちら意図的ではなく、単語を抜かしてしまっていました...
修正します🙏
| setup({ logger }) { | ||
| logger.info('時間が掛かる処理を開始します…'); | ||
| // 何らかの時間が掛かる処理… | ||
| 'config:setup'({ logger }) { | ||
| logger.info('Starting long process…'); | ||
| // 何らかの長い処理… | ||
| }, |
There was a problem hiding this comment.
ここはフックの中身はそのままで良さそうです（logger.info の引数も訳し忘れてしまっています）
| 上記の例では、指定した`info`レベルのメッセージを含むログが出力されます。 | ||
|
|
||
| ```shell | ||
| [long-process-plugin] Starting long process… |
There was a problem hiding this comment.
ここも https://github.com/withastro/starlight/pull/3709/changes#r2805005376 の内容に一致するよう訳す必要がありそうです
|
|
||
| **type:** `(lang: string) => I18nT` | ||
|
|
||
| BCP-47言語タグを指定して`useTranslations()`を呼び出すと、その言語のUI文字列にアクセスするためのユーティリティ関数が生成されます。`useTranslations()`は、Astroコンポーネントで利用可能な`Astro.locals.t()` APIと同等のものを返します。利用可能なAPIの詳細については、[“UI翻訳の使用“](/ja/guides/i18n/#ui翻訳を使用する)ガイドを参照してください。 |
There was a problem hiding this comment.
nitpick ですが「API」直前のスペースは不要そうです:
| BCP-47言語タグを指定して`useTranslations()`を呼び出すと、その言語のUI文字列にアクセスするためのユーティリティ関数が生成されます。`useTranslations()`は、Astroコンポーネントで利用可能な`Astro.locals.t()` APIと同等のものを返します。利用可能なAPIの詳細については、[“UI翻訳の使用“](/ja/guides/i18n/#ui翻訳を使用する)ガイドを参照してください。 | |
| BCP-47言語タグを指定して`useTranslations()`を呼び出すと、その言語のUI文字列にアクセスするためのユーティリティ関数が生成されます。`useTranslations()`は、Astroコンポーネントで利用可能な`Astro.locals.t()`APIと同等のものを返します。利用可能なAPIの詳細については、[“UI翻訳の使用“](/ja/guides/i18n/#ui翻訳を使用する)ガイドを参照してください。 |
|
レビューいただきありがとうございます！ また、一旦PR分けたのですが、おっしゃる通りi18nとリンクの繋がりがあるので、こちらのPRの内容がLGTMになり次第、i18nのPRに合流させようかと思っていましたが、i18nとの相互リンクの繋がり考えると、Shinyaさんからのレビュー項目修正次第、すぐi18nPRに合流させてから確認いただいた方がスムーズでしょうか🤔 |
morinokami
left a comment
morinokami
left a comment
There was a problem hiding this comment.
こちらの内容は LGTM ですので、i18n の方にマージして CI を通すなど残りの作業をお願いします 🙏
|
ありがとうございます！i18n編集PRにmergeしたため、こちらのPRをcloseします🙏 Closing this PR as it has been merged into the following PR: |
3 participants
Description
Update
docs/ja/reference/plugin.mdxto match the English version.Relational Commits:
Additional Info
This PR contains links to
/ja/guides/i18n/#ui翻訳を使用する, an anchor that will be introduced by #3703. So, depending oni18n.mdxeach other.I'd appreciate it if you could review this when you have time🙏