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

[ リンクコンポーネント ] 空の a タグはアクセシビリティやSEO面でよろしくないので対応 #2199

Open
kurudrive opened this issue Sep 3, 2024 · 1 comment

Comments

@kurudrive
Copy link
Member

#2163 (comment)

@mtdkei
Copy link
Contributor

mtdkei commented Sep 4, 2024

@kurudrive

ちょっと色々変更することになりそうなので以下のようにプルリクをわけようと思います。
#2205 のプルリクで順次確認する流れにしました

  1. リンクコンポーネントにEdit link設定を追加
    aria-label、visually-hidden、また、アクセシビリティやSEO面でrel属性の設定もできるようにする予定です。
    また、リンクコンポーネントにEdit link設定を追加するブランチでは、挙動確認のため空のaタグが確認できたブロックのうち一つにリンクコンポーネントの内容を反映します。
  2. 空のaタグが確認できたブロックにリンクコンポーネントの内容を反映
  3. 空のaタグではないブロックでもaria-labelを使っているので1のリンクコンポーネントの内容を反映

これら全てが終わったらdevelopブランチに入れるようにしようかと思ってます。

対象ブロック

  • Outer
  • スライダー
  • グリッドカラム
  • グループ
  • カバー
  • カラム

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants