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

i18n(ja): Update nav.ts #2139

Merged
merged 1 commit into from Dec 6, 2022
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
17 changes: 13 additions & 4 deletions src/i18n/ja/nav.ts
@@ -1,12 +1,17 @@
import { NavDictionary } from '../translation-checkers';

export default NavDictionary({
// Start Here
Copy link
Contributor Author

Choose a reason for hiding this comment

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

他とスタイルを揃えるため、コメントで英語版のセクション名を追加しました。

startHere: 'スタートガイド',
'getting-started': 'はじめに',
install: 'インストール',
'editor-setup': 'エディタのセットアップ',
migrate: 'アップグレードガイド',

// Tutorials
tutorials: 'チュートリアル',
'blog-tutorial': 'ブログを作る',

// Core Concepts
coreConcepts: 'コアコンセプト',
'concepts/why-astro': 'Astroを選ぶ理由',
Expand All @@ -19,9 +24,9 @@ export default NavDictionary({
'core-concepts/astro-components': 'Astroコンポーネント',
'core-concepts/astro-pages': 'ページ',
'core-concepts/layouts': 'レイアウト',
'guides/markdown-content': 'Markdown',
'guides/markdown-content': 'MarkdownとMDX',
'core-concepts/routing': 'ルーティング',
'guides/imports': '静的アセット',
'guides/imports': 'インポート',
Comment on lines -22 to +29
Copy link
Contributor Author

Choose a reason for hiding this comment

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

この二つについては、対応するページのタイトルだけでなくページ内容も割と変わっているため、変更するか迷いました(もとのままとしたほうが翻訳済みページの内容とは合致するため)が、古いまま残り続けてしまう可能性を懸念して一旦英語版に合わせました。問題があればもとに戻そうと思います。

なお、両変更がされたのは以下の PR です:

Copy link
Contributor

Choose a reason for hiding this comment

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

MDXに関してはページを更新した際に直そうかと思っていましたが、更新された方が気づかず、残り続けてしまうのはその通りだと思うので、英語版に合わせるので良いと思います。
こうしておけば、日本語版を更新するまでの間にアクセスしたユーザーが、MDXについて記載がないことに気づいて英語版を確認してもらえるかもしれないという期待もできますし。

'core-concepts/endpoints': 'エンドポイント',
'guides/data-fetching': 'データフェッチ',
'guides/deploy': 'デプロイ',
Expand All @@ -31,17 +36,20 @@ export default NavDictionary({
features: 'ガイド',
'guides/integrations-guide': 'インテグレーション',
'core-concepts/framework-components': 'UIフレームワーク',
'guides/server-side-rendering': 'サーバーサイドレンダリング(SSR)',
'guides/client-side-scripts': 'スクリプトとイベントハンドリング',
'guides/styling': 'CSSとスタイル',
'guides/server-side-rendering': 'サーバーサイドレンダリング(SSR)',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

実際の表示順に合わせるために SSR の行を移動しました。

'guides/content': 'コンテンツの執筆',
'guides/cms': 'CMSとの接続',
'guides/images': '画像',
'guides/fonts': 'フォント',
'core-concepts/sharing-state': '状態の共有',
'guides/rss': 'RSS',
'guides/testing': 'テスト',

// Configuration
configuration: '設定',
'guides/configuring-astro': 'Astroコンフィグファイル',
'guides/configuring-astro': 'Astroの設定ファイル',
Comment on lines -44 to +52
Copy link
Contributor Author

Choose a reason for hiding this comment

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

こちらはもとのままでも良かったかもしれませんが、個人的な感覚として日本語で話すときは「設定ファイル」という言葉のほうが普通な気がしたこと、そして対応するページ内で "The Astro Config File" という英語が「Astroの設定ファイル」と訳されていたことから、nav の方も同じ言葉使いに改めました。

Copy link
Contributor

Choose a reason for hiding this comment

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

良さそうです!nav.tsはコンテンツが翻訳される前に訳しているものもあり、見直しが必要だと思っていました。気づいていただきありがとうございます!

'guides/typescript': 'TypeScript',
'guides/aliases': 'importエイリアス',
'guides/environment-variables': '環境変数',
Expand All @@ -54,5 +62,6 @@ export default NavDictionary({
'reference/integrations-reference': 'インテグレーションAPI',
'reference/adapter-reference': 'アダプターAPI',
'reference/directives-reference': 'テンプレートディレクティブ',
'reference/error-reference': 'エラーリファレンス',
'guides/publish-to-npm': 'npmへの公開',
});