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

Feat: aria-labelledby相当の機能がSwitchにほしいです #564

Open
mj-hd opened this issue Jun 24, 2024 · 1 comment
Open

Feat: aria-labelledby相当の機能がSwitchにほしいです #564

mj-hd opened this issue Jun 24, 2024 · 1 comment
Assignees

Comments

@mj-hd
Copy link

mj-hd commented Jun 24, 2024

必要だと思う機能の内容を書いてください

他の構造的に離れた要素でラベルを提供しているUIを表現するためaria-labelledbyかそれと同等の機能がほしいです

この機能が必要と考えられる理由を書いてください

APGによると、switchロールはaria-labelや視覚可能なテキストコンテンツの代わりにaria-labelledby属性を指定することができるとあります。
https://www.w3.org/WAI/ARIA/apg/patterns/switch/

現在の実装だと、labelか、childrenしか受け取りません。

| // children か label は片方が必須

この機能があった場合にどういう動きをするかの例を書いてください

react-ariaのuseSwitchのariaPropsにこれらの値が渡り、いろいろなユーザエージェントがこれを読み取れる状態。

また、他にもaria属性はいくつかサポートされていて、aria-description/aria-describedbyなどの追加テキスト情報を提供できるものや、aria-controlsによる操作対象のUIを指定できるものがありそうです。


Switch control should expose aria-labelledby props

According to APG, switch role may have an aria-labelledby attr instead of aria-label.
https://www.w3.org/WAI/ARIA/apg/patterns/switch/

In the current implementation, the Switch component only exposes aria-label and children as text content.

| // children か label は片方が必須

(...and, there are more aria props, aria-description and aria-describedby, aria-details attrs to represent additional text content, aria-controls to represent content controlled by the switch.)

@mj-hd mj-hd changed the title Switch control should expose aria-labelledby props Feat: aria-labelledby相当の機能がSwitchにほしいです Jun 24, 2024
@toshusai toshusai self-assigned this Jul 1, 2024
@toshusai
Copy link
Contributor

toshusai commented Jul 1, 2024

Issueの作成およびフィードバックをいただき、誠にありがとうございます。

v4.0.0の変更により、Switchのinput要素に対して任意の属性を付与できるようになる予定です。そのため、申し訳ありませんが現在 v3 のAPIに変更を加える事は検討されていません。

v4.0.0のリリース時にご確認いただければ幸いです。また、現在beta版もリリースしておりますので、そちらでもご確認いただく事ができます。

参考までに、下記はv4.0.0のSwitchのコードになります。
https://github.com/pixiv/charcoal/blob/6ce3eaf89f87ceec7cd0b5031c3f6227096c289d/packages/react/src/components/Switch/index.tsx

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

No branches or pull requests

2 participants