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 · 4 comments

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属性の設定もできるようにする予定です。(aria-labelを残すのは3の以下理由によるもの。)
  2. 空のaタグが確認できたブロックに1のリンクコンポーネントの内容を反映
  3. 空のaタグではないブロックでもaria-labelを使っているので1のリンクコンポーネントの内容を反映(もしかしたらないかもれない)

対象ブロック

もしかしたら他にもあるかもしれませんが把握しているものは以下です。

VK Blocks Pro

コアブロック

  • グループ
  • カバー
  • カラム

@mtdkei
Copy link
Contributor

mtdkei commented Dec 6, 2024

@MasayaMORIMOTO @akito-38
もし可能でしたら、こちらのissueで #2362 のFiles Changedと同じように他のブロックでも対応いただけますでしょうか?「対象ブロック」の中で未チェックのブロックをやっていただけたら大変嬉しいです。

@MasayaMORIMOTO
Copy link
Collaborator

@mtdkei
承知しました。
同様のファイル変更内容で「対象ブロック」のものを修正していきます。

@sysbird
Copy link
Member

sysbird commented Dec 23, 2024

仕様の補足です

  • Accesibility link description がある場合は、aria-label 属性はいらない
  • target="" 空の場合は出力しない
  • rel="" 空の場合は出力しない

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

5 participants