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

Use HTML spec compliant radio element #110

Closed
sophie-cluml opened this issue Jul 1, 2024 · 2 comments · Fixed by #151
Closed

Use HTML spec compliant radio element #110

sophie-cluml opened this issue Jul 1, 2024 · 2 comments · Fixed by #151
Assignees

Comments

@sophie-cluml
Copy link
Contributor

sophie-cluml commented Jul 1, 2024

Background

#98 (comment)

Tasks

기존에 있는 코드 중 radio 버튼에 관한 코드를 HTML 5 compliant 하게 변경합니다.

  • 단순히 <div>가 아닌 <input type="radio"> 으로 변경
  • <div>로 구현하다고 하더라도 role="radio" 속성을 넣어 줍니다.
@sophie-cluml
Copy link
Contributor Author

@div-seungha 제가 이슈를 대신 생성하였습니다. 이슈를 해결하는 데에 더 도움이 되는 내용이 있다면 추가해주시면 감사하겠습니다.

@sehkone
Copy link
Contributor

sehkone commented Jul 1, 2024

<div>로 구현되어 있는 코드를 <input type="radio">로 바꿀 필요는 없으므로 위 Tasks를 아래와 같이 수정합니다.

Tasks

<input type="radio">를 사용하지 않고 <div>로 구현되어 있으나 radio button으로 기능하는 요소에 대해서는 role="radio"를 추가하여 HTML 5 표준에 부합되도록 한다.

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

Successfully merging a pull request may close this issue.

3 participants