kufu / smarthr-ui

React components for creating SmartHR applications.
https://story.smarthr-ui.dev
MIT License
879 stars 141 forks source link

fix(LanguageSwitcher): スクリーンリーダーで各言語のリンクを正しく読み上げられるようにlang属性を追加 #5002

Closed daiHash closed 1 month ago

daiHash commented 1 month ago

関連URL

https://smarthr.atlassian.net/browse/SHRUI-1177

Storybook WCAG Reference

概要

言語切替メニューのリンクのテキストが各言語の言葉のため、pageに設定されている言語によってスクリーンリーダーで読み上げるときに発音がおかしかったり、読み上げられない場合がります

変更内容

確認方法

現在の状態の確認は

pkg-pr-new[bot] commented 1 month ago

Open in Stackblitz

pnpm add https://pkg.pr.new/kufu/smarthr-ui@5002

commit: 07206b4

Qs-F commented 1 month ago

些細な話なのですが、Conventional commitsではスコープ切る場合、型の後方に (hogehoge) でつけると良いと書いてあるので、[]で囲うよりも fix(LanguageSwitcher): ... の形で書くほうがいいかもです〜!

s-sasaki-0529 commented 1 month ago

ひえ、CIが回らないゾ

Qs-F commented 1 month ago

あ〜〜〜 そうか、メンテナーに入ってないからCIが回らないのか… 😇