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/#187 노래(킬링파트) 조회 페이지 웹 접근성 추가 #189

Merged
merged 6 commits into from
Aug 3, 2023

Conversation

ukkodeveloper
Copy link
Collaborator

📝작업 내용

  • 페이지별로 h1 태그를 추가하여 웹접근성을 향상시키기 위한 SRHeading 컴포넌트 추가
  • 화면엔 보이지 않고 안내 메세지를 읽어주기 위한 SRAlert 컴포넌트 추가
  • 노래 조회 페이지 SongPage 웹 접근성 요소 추가

작업내용

💬리뷰 참고사항

원활한 리뷰를 위해 전달하고 싶은 맥락(특정 파일, 디렉터리 등등)
리뷰어가 특별히 봐주었으면 하는 부분

웹 접근성 코드만 수정하였기 때문에 피드백 주시면 바로 반영하겠습니다.

  • 더 세련된 aria 설정이 있는 경우
  • 굳이 필요없는 스크린 리더 안내가 있을 경우
  • 추가로 필요한 안내가 있을 경우

피드백 주세요.

#️⃣연관된 이슈

연관된 이슈 번호를 모두 작성

close #187

@ukkodeveloper ukkodeveloper self-assigned this Aug 2, 2023
@ukkodeveloper ukkodeveloper added [ 🌞 FE ] 프론트엔드 크루들의 빛나는 개발 이야기 하나둘셋 호! ✨ Feat 꼼꼼한 기능 구현 중요하죠 labels Aug 2, 2023
@github-actions
Copy link

github-actions bot commented Aug 2, 2023

Unit Test Results

114 tests   114 ✔️  4s ⏱️
  34 suites      0 💤
  34 files        0

Results for commit e60ac14.

♻️ This comment has been updated with latest results.

@cruelladevil cruelladevil self-requested a review August 3, 2023 07:47
@ukkodeveloper ukkodeveloper requested review from Creative-Lee and cruelladevil and removed request for cruelladevil and Creative-Lee August 3, 2023 07:47
@ukkodeveloper ukkodeveloper merged commit 38c4311 into main Aug 3, 2023
@ukkodeveloper ukkodeveloper deleted the feat/#187 branch August 3, 2023 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[ 🌞 FE ] 프론트엔드 크루들의 빛나는 개발 이야기 하나둘셋 호! ✨ Feat 꼼꼼한 기능 구현 중요하죠
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FEAT] 음악 조회 페이지 접근성 개선
2 participants