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

색깔 접근성 문제 해결 및 스토리북 접근성 테스트 비활성화 해제 #199

Merged
merged 5 commits into from
Jan 23, 2025

Conversation

SamTheKorean
Copy link
Contributor

@SamTheKorean SamTheKorean commented Jan 14, 2025

체크리스트

  • 이슈가 연결되어 있나요?
  • 배포 후 브라우저 콘솔에 경고나 오류가 있나요?

요약

더 이상 접근성 테스트를 비활성화 할 일이 없으므로 disabled를 false로 바꾸는 대신 관련 매개변수 코드를 삭제.

@SamTheKorean SamTheKorean self-assigned this Jan 14, 2025
@SamTheKorean SamTheKorean requested a review from a team as a code owner January 14, 2025 01:18
@SamTheKorean SamTheKorean linked an issue Jan 14, 2025 that may be closed by this pull request
@SamTheKorean SamTheKorean changed the title update : remove disable param for acceessibility test 스토리북 접근성 테스트 비활성화 해제 Jan 14, 2025
@SamTheKorean SamTheKorean changed the title 스토리북 접근성 테스트 비활성화 해제 색깔 접근성 문제 해결 및 스토리북 접근성 테스트 비활성화 해제 Jan 14, 2025
@SamTheKorean
Copy link
Contributor Author

해당 pr에서 접근성 테스트 비활성화를 해제하고 보니 아직 실패하고 있는 접근성 테스트가 있습니다. 해당 pr에서 실패하는 테스트 해결하고 비활성화도 해제하겠습니다.

@SamTheKorean
Copy link
Contributor Author

@yolophg 접근성 테스트 확인해보니 푼 문제 수 색깔과 사이드바 배경과의 대조도 선명하지 못해 테스트가 실패하고 있습니다! 디자인 변경이 필요할 것 같습니다!
Screenshot 2025-01-14 at 7 33 05 PM

@SamTheKorean
Copy link
Contributor Author

SamTheKorean commented Jan 15, 2025

@yolophg 접근성 테스트 확인해보니 푼 문제 수 색깔과 사이드바 배경과의 대조도 선명하지 못해 테스트가 실패하고 있습니다! 디자인 변경이 필요할 것 같습니다! Screenshot 2025-01-14 at 7 33 05 PM

정정하겠습니다. 해당 이슈는 테스트 실패의 원인은 아니고 다른 걸 보고 있었네요..

@SamTheKorean
Copy link
Contributor Author

SamTheKorean commented Jan 15, 2025

@yolophg
Screenshot 2025-01-14 at 8 16 05 PM
이 부분은 디자인 변경이 필요한 것 같아 공유드립니다. 현재 테이블에 보시면 hard의 색깔과 연보라색 색깔의 대조가 불분명해서 접근성 테스트가 실패하고 있습니다! 이 이슈 제외하고는 다른 접근성 테스트는 통과하고 있습니다.

@yolophg
Copy link

yolophg commented Jan 15, 2025

@yolophg Screenshot 2025-01-14 at 8 16 05 PM 이 부분은 디자인 변경이 필요한 것 같아 공유드립니다. 현재 테이블에 보시면 hard의 색깔과 연보라색 색깔의 대조가 불분명해서 접근성 테스트가 실패하고 있습니다! 이 이슈 제외하고는 다른 접근성 테스트는 통과하고 있습니다.

확인 감사합니다. 관련 티켓 따로 파두었습니다, 이번 스프린트에 같이 작업하여 업데이트 될 예정입니다.

@SamTheKorean
Copy link
Contributor Author

감사합니다! 남은 이슈까지 해결되면 현재 pr마무리하겠습니다!

@yolophg
Copy link

yolophg commented Jan 19, 2025

@yolophg Screenshot 2025-01-14 at 8 16 05 PM 이 부분은 디자인 변경이 필요한 것 같아 공유드립니다. 현재 테이블에 보시면 hard의 색깔과 연보라색 색깔의 대조가 불분명해서 접근성 테스트가 실패하고 있습니다! 이 이슈 제외하고는 다른 접근성 테스트는 통과하고 있습니다.

해당 이슈 컬러 업데이트 완료되었으니 코드에서 업데이트 후 pr 닫아주시면 될 듯 합니다 :). 감사합니다!
=> 관련 디자인 이슈

@SamTheKorean
Copy link
Contributor Author

@DaleSeo 도움 감사드립니다! 말씀하신대로 aria-label에 관련한 문제들은 불필요한 label을 삭제해주면서 큰 코드 변경없이 해결하였는데
Screenshot 2025-01-20 at 9 07 08 PM 나머지 한 개 오류에서는 주변 html과 css 코드 수정이 불가피할 것 같습니다..! 이 부분도 sementic 구조에 대한 이해가 부족한 상태에서 코딩해서 발생한 것 같습니다.

@SamTheKorean
Copy link
Contributor Author

생각해보니 이 부분도 큰 변경사항 없이 해결할 수 있을 것 같습니다!

@DaleSeo 도움 감사드립니다! 말씀하신대로 aria-label에 관련한 문제들은 불필요한 label을 삭제해주면서 큰 코드 변경없이 해결하였는데 Screenshot 2025-01-20 at 9 07 08 PM 나머지 한 개 오류에서는 주변 html과 css 코드 수정이 불가피할 것 같습니다..! 이 부분도 sementic 구조에 대한 이해가 부족한 상태에서 코딩해서 발생한 것 같습니다.

@SamTheKorean
Copy link
Contributor Author

SamTheKorean commented Jan 21, 2025

ui 변경 없이 접근성 이슈 모두 해결하여서 pr 다시 열었습니다! 덕분에 sementic tag의 올바른 사용에 대해서 공부할 수 있었습니다. @DaleSeo 님 말씀대로 남용하고 있던 접근성 관련 요소들을 지움으로써 해결하였습니다.

Copy link
Contributor

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

와, 드디어 접근성 위반에서 해방되었네요! 💯

src/components/Card/Card.tsx Show resolved Hide resolved
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 this pull request may close these issues.

색깔 접근성 문제 해결 구현
3 participants