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

[Outer] アクセシビリティ対応テキストがある場合は、aria-label 属性はいらないかもしれない #2380

Closed
sysbird opened this issue Dec 19, 2024 · 5 comments
Assignees

Comments

@sysbird
Copy link
Member

sysbird commented Dec 19, 2024

本日12/19 のお知らせにあった、リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定追加(ここではOuterのみ) #2362 について、リンクツールバーに a タグのアクセシビリティや SEO に対応できるよう設定追加 (グリッドカラム) #2367 を確認中に絡んで確認しました

  • Accesibility link description がある場合は、aria-label 属性はいらないのでは?
  • target="" 空の場合は出力しないほうがいいかもしれない
  • rel="" 空の場合は出力しないほうがいいかもしれない
@mtdkei
Copy link
Contributor

mtdkei commented Dec 20, 2024

@sysbird
ありがとうございます!

今の所、他のブロックが aria-label に依存している現状のため、Outer においても暫定的に aria-label を残している状態でした。ただおっしゃる通りOuterブロックではいらないので削除の方向で考えてみます。

また、target="" と rel="" においてはおっしゃる通りです。こちらはリンクコンポーネントの件のため、上記とは別方向で処理を進めていくことになるかと思います。

上記2点をもしやってくださる方がいればお願いしたいですが、急ぎであれば対応します。
対応します。

@sysbird
Copy link
Member Author

sysbird commented Dec 20, 2024

@mtdkei
ありがとうございます!
ぜんぜん急ぎじゃないですーー

@sysbird
Copy link
Member Author

sysbird commented Dec 20, 2024

@mtdkei
aria-label、target=""、rel="" はリンクコンポーネントでの対応になりますか?
そうしますと、各ブロックでアクセシビリティ対応テキストを対応している途中ですので、それが済んでからのほうがよいでしょうかね

@mtdkei
Copy link
Contributor

mtdkei commented Dec 20, 2024

@sysbird
ありがとうございます。今確認している感じだと各ブロック対応の内容だったのでOuter側で対応してみてます!

@sysbird
Copy link
Member Author

sysbird commented Dec 23, 2024

対応ありがとうございました
マージされたので閉じます

@sysbird sysbird closed this as completed Dec 23, 2024
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