Skip to content

Conversation

@Yusaku01
Copy link
Contributor

Description (required)

This PR's change is based on the following the file:

Related issues & labels (optional)

  • Closes #
  • Suggested label: i18n

I'd appreciate it if you could review this when you have time🫡

@github-actions github-actions bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Jan 21, 2026
@astrobot-houston
Copy link
Contributor

astrobot-houston commented Jan 21, 2026

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
ja/guides/fonts.mdx Localization added, will be marked as complete.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@netlify
Copy link

netlify bot commented Jan 21, 2026

Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit a36ec83
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/69719a58dd986000088ec8fa
😎 Deploy Preview https://deploy-preview-13080--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Yusaku01 Yusaku01 changed the title i18n(ja): add the guide file fonts.mdx i18n(ja): add the new guide file fonts.mdx Jan 21, 2026
Copy link
Contributor

@morinokami morinokami left a comment

Choose a reason for hiding this comment

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

対応ありがとうございます!全体的に読みやすくほとんど問題なさそうでしたが、細かい点についていくつかコメントしましたので、ご確認をお願いします 🙏

sidebar:
label: Fonts
description: >-
Astroウェブサイトにカスタムフォントを追加したい時はありませんか?FontsourceでGoogle Fontsを使用するか、お好みのフォントを追加しましょう。
Copy link
Contributor

Choose a reason for hiding this comment

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

少し細かいのですが、前半の原文は

Looking to add some custom typefaces to an Astro website?

となっており、Looking to のニュアンスと「したい時はありませんか?」がちょっとズレている気がしました。ストレートに

Suggested change
Astroウェブサイトにカスタムフォントを追加したい時はありませんか?FontsourceでGoogle Fontsを使用するか、お好みのフォントを追加しましょう。
Astroウェブサイトにカスタムフォントを追加したいですか?FontsourceでGoogle Fontsを使用するか、お好みのフォントを追加しましょう。

のような感じでいい気がしました。

Copy link
Contributor Author

@Yusaku01 Yusaku01 Jan 22, 2026

Choose a reason for hiding this comment

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

レビューいただきありがとうございます!
修正対応させていただきました!

Comment on lines 104 to 105
ウェブサイトのレンダリング時間を最適化するには、初期ページ表示に必要なフォントを事前レンダリングするとよいでしょう。
詳細と使用方法については、[Fontsourceのフォントプリロードガイド](https://fontsource.org/docs/getting-started/preload)を参照してください。
Copy link
Contributor

Choose a reason for hiding this comment

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

こちらも細かいですが改行により不要なスペースが生じてしまっているので直したほうが良さそうです:

Image
Suggested change
ウェブサイトのレンダリング時間を最適化するには、初期ページ表示に必要なフォントを事前レンダリングするとよいでしょう。
詳細と使用方法については、[Fontsourceのフォントプリロードガイド](https://fontsource.org/docs/getting-started/preload)を参照してください。
ウェブサイトのレンダリング時間を最適化するには、初期ページ表示に必要なフォントを事前レンダリングするとよいでしょう。詳細と使用方法については、[Fontsourceのフォントプリロードガイド](https://fontsource.org/docs/getting-started/preload)を参照してください。

Copy link
Contributor Author

@Yusaku01 Yusaku01 Jan 22, 2026

Choose a reason for hiding this comment

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

レビューいただきありがとうございます!
修正対応させていただきました!

Copy link
Contributor

@morinokami morinokami left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@morinokami morinokami merged commit 4ffbed4 into withastro:main Jan 22, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants