Skip to content

add 2.4.11#60

Merged
NakajimaTakuya merged 3 commits intofeature/wcag2_2from
feature/wcag2_2--2_4_11
Dec 19, 2024
Merged

add 2.4.11#60
NakajimaTakuya merged 3 commits intofeature/wcag2_2from
feature/wcag2_2--2_4_11

Conversation

@NakajimaTakuya
Copy link
Copy Markdown

#57
https://www.w3.org/TR/WCAG22/#focus-not-obscured-minimum の落とし込み

内容が実装寄りになるので統合位置はそめヲの提案通り実装-インタラクション-level1-フォーカスインジケータとした。

2.4.11ではキーボードフォーカスを動かさずに覆い被さってる状態を解除できる(例えばescキーとか)ならまぁOKという文脈もあるが、閉じたいコンテンツにフォーカスを移さずにキー操作で閉じるといった解決方法をとるケースが思いつかないので一旦スルーした

Copy link
Copy Markdown
Contributor

@shimaokk shimaokk left a comment

Choose a reason for hiding this comment

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

コメントしました!


##### ヒント:フォーカスを受け取ったUIが追従コンテンツによって隠されないようにする

ページをタブで下に移動すると、フォーカスの当たった要素が表示領域に収まるようにスクロールを伴いますが、追従ヘッダ等がある場合、それによってフォーカスされたUIが隠れてしまうケースがあります。 このような場合、`scroll-padding`プロパティを設定することで要素の重なりを回避することができます。
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

タブはキーボード操作のことよね?
他でEscapeキーのように使ってるから、Tabキーみたいにして統一したほうがよいかも。

あと、句読点の後にスペースが入ってます。

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@shimaokk
修正しましたん
3d15107


##### ヒント:Cookieの利用同意を求めるモーダルダイアログにフォーカスを移動する

Cookie利用同意のポップアップはしばしばダイアログとして実装され、ページの読み込み後すぐに表示されます。 このダイアログを背面と対話可能なモードレスダイアログとして実装すると、背面のUIがダイアログに完全に隠され、操作が困難になる可能性があります。 この問題を回避するため、ダイアログはモーダルとして実装し、閉じるまでフォーカスが外側のコンテンツに移動しないようにすることが推奨されます。
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

句読点の後にスペースが入ってます。

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@shimaokk
修正しましたん
df12e19

Copy link
Copy Markdown
Contributor

@shimaokk shimaokk left a comment

Choose a reason for hiding this comment

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

LGTM!

@NakajimaTakuya NakajimaTakuya merged commit c593ae9 into feature/wcag2_2 Dec 19, 2024
@NakajimaTakuya NakajimaTakuya deleted the feature/wcag2_2--2_4_11 branch December 19, 2024 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants