Skip to content

Conversation

@ef81sp
Copy link
Contributor

@ef81sp ef81sp commented Sep 21, 2025

関連Issue

レビュワーに見てもらいたい箇所

翻訳チェックリスト

  • html要素にlang="ja"属性を付与する
  • h1要素の直下に翻訳元リビジョンを記載する

@netlify
Copy link

netlify bot commented Sep 21, 2025

Deploy Preview for waic-apg ready!

Name Link
🔨 Latest commit c252a0a
🔍 Latest deploy log https://app.netlify.com/projects/waic-apg/deploys/692d91710e39c1000835eb97
😎 Deploy Preview https://deploy-preview-160--waic-apg.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.

@ef81sp ef81sp added this to the 1st release milestone Sep 21, 2025
@ef81sp ef81sp self-assigned this Sep 21, 2025
@ef81sp ef81sp requested a review from Copilot September 21, 2025 04:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR translates the Radio Group pattern pages and related examples into Japanese and updates coverage metadata and report content to reflect the new titles and references.

  • Localizes three radio-related HTML pages (pattern and two examples), including headings, instructional text, and ARIA labels.
  • Updates coverage CSVs and the coverage report HTML to align counts, labels, and links with the translated content.

Reviewed Changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 10 comments.

Show a summary per file
File Description
content/patterns/radio/radio-group-pattern.html Japanese translation of the Radio Group pattern documentation.
content/patterns/radio/examples/radio.html Japanese translation of the roving tabindex radio example, including UI strings and accessibility labels.
content/patterns/radio/examples/radio-rating.html Japanese translation of the rating radio example; updated example section content and labels.
content/patterns/radio/examples/radio-activedescendant.html Japanese translation of the aria-activedescendant radio example, including tables and accessibility notes.
content/about/coverage-and-quality/role-coverage.csv Synchronizes role coverage counts and example titles with translations.
content/about/coverage-and-quality/prop-coverage.csv Synchronizes property/state coverage references with translated example titles.
content/about/coverage-and-quality/coverage-and-quality-report.html Updates report counts and example names/links to match translations.
Comments suppressed due to low confidence (1)

content/patterns/radio/examples/radio-activedescendant.html:1

  • The preceding line refers to aria--activedescendant (double hyphen). Correct to aria-activedescendant in the text to avoid confusion.
<!DOCTYPE html>

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ef81sp ef81sp moved this from 未着手 to レビュー中 in APG日本語訳 Sep 25, 2025
@ef81sp ef81sp force-pushed the rikegami/radio-ja branch from d805dce to 7eaa648 Compare November 1, 2025 12:01
Copy link
Contributor Author

@ef81sp ef81sp left a comment

Choose a reason for hiding this comment

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

patternだけ見た

Copy link
Contributor Author

@ef81sp ef81sp left a comment

Choose a reason for hiding this comment

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

途中)radio-activedecendantを見ました

Copy link
Contributor Author

@ef81sp ef81sp left a comment

Choose a reason for hiding this comment

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

このPRの全ファイルを見た
これから翻訳漏れの翻訳と、レビュー内容の反映をします

ef81sp and others added 20 commits January 24, 2026 19:58
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@netlify
Copy link

netlify bot commented Jan 24, 2026

Deploy Preview for waic-apg ready!

Name Link
🔨 Latest commit a398a97
🔍 Latest deploy log https://app.netlify.com/projects/waic-apg/deploys/6974d2fbe0d6d80008e0a9c4
😎 Deploy Preview https://deploy-preview-160--waic-apg.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.

ef81sp and others added 3 commits January 24, 2026 22:33
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
PR#160 レビューコメント対応

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
sourceCode.add が 'ex1' を参照しているため原文と一致させる

PR#160 レビューコメント対応

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@ef81sp
Copy link
Contributor Author

ef81sp commented Jan 24, 2026

主査レビューへ

@ef81sp ef81sp moved this from レビュー中 to 主査レビュー待ち in APG日本語訳 Jan 24, 2026
@ef81sp ef81sp requested a review from caztcha January 24, 2026 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 主査レビュー待ち

Development

Successfully merging this pull request may close these issues.

Radio Group

2 participants