yamada-ui / yamada-ui

React UI components of the Yamada, by the Yamada, for the Yamada built with React and Emotion.
https://yamada-ui.com
MIT License
810 stars 216 forks source link

Elements must only use allowed ARIA attributes #591

Closed hirotomoyamada closed 9 months ago

hirotomoyamada commented 10 months ago

Which package has an issue?

@yamada-ui/react

Package Version

1.1.2

Description

When setting isReadonly to DatePicker or MouthPicker components, aria-readonly is assigned to unnecessary elements.

Target Components

Link to Reproduction

  1. Set isReadonly to Switch or Select components. 2. Assigned aria-readonly to element. 3. aria-readonly is assigned to unnecessary elements.

Steps to Reproduce

  1. Set isReadonly to Switch or Select components.
  2. Assigned aria-readonly to element.
  3. aria-readonly is assigned to unnecessary elements.

Browser

Chrome

Operating System

Do you know how to fix the issue?

Yes

Are you willing to participate in fixing this issue and create a pull request with the fix?

None

Additional Information

No response

ISAWASHUN commented 10 months ago

@hirotomoyamada

Nice to meet you, I'm currently a college student doing Rails as an intern, and I found this issue through Zenn's article. I have no practical experience with Next.js, but I would like to try OSS. May I work on this issue?

日本語訳

初めまして、現在大学生でインターンでRailsを行っているものです。Zennの記事を見てこのissueを発見しました。 Next.jsは実務で経験がないですが、OSS活動に挑戦したいと思っています。 このissueに取り組んでもよろしいでしょうか。

hirotomoyamada commented 10 months ago

@ISAWASHUN

こんにちは。はじめまして!ぜひ、OSSに参加してみましょう!このイシューを取り組んでみましょう!

Let's Try!!!

ISAWASHUN commented 10 months ago

@hirotomoyamada

ありがとうございます! 取り組みます!

hirotomoyamada commented 10 months ago

@ISAWASHUN

こんにちは!

584

574

545

このPRを参考にすると良いと思います。やっていることは、同じ作業です。

また、aria-readonlyが不要な要素を発見するには、各コンポーネントのStoryAccessibilityを見ると良いでしょう。

ISAWASHUN commented 10 months ago

@hirotomoyamada

アドバイスありがとうございます! 今日の夜から本格的に進めていきます!

hirotomoyamada commented 9 months ago

@ISAWASHUN

こんにちは!

このイシューの状況は、どうですか?

明日、このイシューのアサインを初期の状態に戻します。

このイシューを取り組んでおり、延長する場合は、こちらに連絡をください。

ISAWASHUN commented 9 months ago

@hirotomoyamada

卒論と上京準備でバタバタしてたので、取り組めてませんでした。 ご迷惑をお掛けして申し訳ないです。外してもらって大丈夫です。 申し訳ございません。

apple-yagi commented 9 months ago

@hirotomoyamada

Hi there! I'm interested in working on this issue. Is it okay if I take on this task?

hirotomoyamada commented 9 months ago

@apple-yagi

Hi, thanks for the comment!

Yes, please!

I will assign this issue to you, Good luck!