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

search ロールを 2024/10/23 時点の英語版に同期 #24204

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

hiroya-uga
Copy link
Contributor

Description

https://developer.mozilla.org/ja/docs/Web/Accessibility/ARIA/Roles/search_role

Motivation

Additional details

Related issues and pull requests

@hiroya-uga hiroya-uga requested a review from a team as a code owner October 23, 2024 01:02
@hiroya-uga hiroya-uga requested review from mfuji09 and removed request for a team October 23, 2024 01:02
@github-actions github-actions bot added the l10n-ja Issues related to Japanese content. label Oct 23, 2024
Copy link
Contributor

github-actions bot commented Oct 23, 2024

Preview URLs

Flaws (2)

URL: /ja/docs/Web/Accessibility/ARIA/Roles/search_role
Title: ARIA: search ロール
Flaw count: 2

  • broken_links:
    • Can't resolve /ja/docss/Web/HTML/Element/search
    • Can't resolve /ja/docss/Web/HTML/Element/form

(comment last updated: 2024-10-24 08:48:51)

@hiroya-uga hiroya-uga force-pushed the patch-1 branch 2 times, most recently from 4f07538 to 555f62b Compare October 23, 2024 01:10
Copy link
Collaborator

@mfuji09 mfuji09 left a comment

Choose a reason for hiding this comment

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

2件について、確認と修正をお願いします。

@@ -3,7 +3,7 @@ title: "ARIA: search ロール"
slug: Web/Accessibility/ARIA/Roles/search_role
---

検索 (`search`) [ランドマークロール](/ja/docs/Web/Accessibility/ARIA/ARIA_Techniques#landmark_roles)は、ページ、サイト、またはサイトのコレクションの検索に使用されるページのセクションを識別するために使用します。
検索 (`search`)ロールは、ページ、サイト、またはサイトのコレクションの検索に使用されるページのセクションを識別するために使用します。
Copy link
Collaborator

Choose a reason for hiding this comment

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

半角括弧と日本語の間に、半角スペースを入れてください。
同様の問題が他の部分にもありますので、そちらも修正をお願いします。

Suggested change
検索 (`search`)ロールは、ページ、サイト、またはサイトのコレクションの検索に使用されるページのセクションを識別するために使用します。
検索 (`search`) ロールは、ページ、サイト、またはサイトのコレクションの検索に使用されるページのセクションを識別するために使用します。

Copy link
Contributor Author

@hiroya-uga hiroya-uga Oct 24, 2024

Choose a reason for hiding this comment

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

修正いたしました!


## 例

[`<form>`](/ja/docss/Web/HTML/Element/form)が検索フォームの場合、[`form`](/ja/docs/Web/Accessibility/ARIA/Roles/form_role) roleの代わりに`search` roleを使用してください。
Copy link
Collaborator

Choose a reason for hiding this comment

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

他の部分では role をカナ書きしているので、この部分でも合わせてください。

Suggested change
[`<form>`](/ja/docss/Web/HTML/Element/form)が検索フォームの場合、[`form`](/ja/docs/Web/Accessibility/ARIA/Roles/form_role) roleの代わりに`search` roleを使用してください
[`<form>`](/ja/docss/Web/HTML/Element/form) が検索フォームの場合、[`form`](/ja/docs/Web/Accessibility/ARIA/Roles/form_role) ロールの代わりに `search` ロールを使用してください

Copy link
Contributor Author

@hiroya-uga hiroya-uga Oct 24, 2024

Choose a reason for hiding this comment

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

こちらも修正いたしました!

@hiroya-uga
Copy link
Contributor Author

@mfuji09 レビューありがとうございます!
ご指摘いただいた部分修正いたしましたので、改めてご確認いただけますでしょうか。

どうぞよろしくお願いいたします。

Copy link
Collaborator

@mfuji09 mfuji09 left a comment

Choose a reason for hiding this comment

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

修正ありがとうございました。OKです。

@mfuji09 mfuji09 merged commit 7b56d7e into mdn:main Oct 25, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-ja Issues related to Japanese content.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants