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

웹 접근성 개선 #693

Open
wants to merge 3 commits into
base: develop-frontend
Choose a base branch
from
Open

웹 접근성 개선 #693

wants to merge 3 commits into from

Conversation

cys4585
Copy link
Contributor

@cys4585 cys4585 commented Oct 20, 2024

PR의 목적이 무엇인가요?

다락방 생성, 참여 플로우와 모임 생성 플로우에 대한 접근성을 개선했어요. 스크린 리더 사용자의 UX를 개선했어요.

이슈 ID는 무엇인가요?

설명

  1. index.html의 언어를 한국어로 수정했습니다.
  2. 다락방 참여 플로우의 중첩 span를 끊어 읽는 문제를 aria-label로 해결했습니다.
  3. 뒤로가기 버튼 포커싱 안되는 부분을 button 태그 사용하는 컴포넌트로 교체했습니다. 또한 aria-label로 뒤로가기 버튼임을 인지할 수 있도록 했어요.
  4. 모임 생성 플로우의 Label, Input, Validation condition을 연결해서 읽을 수 있도록 했어요. aria-labelledby, aria-describedby를 사용했습니다.
  5. 모임 생성 시 validation 에러를 발견해서 함께 해결했습니다. (날짜에 상관없이 현재 시간을 기준으로만 시간 validation 하는 문제 해결, 하단 버튼으로 인해 입력창 가려지는 문제 해결)

질문 혹은 공유 사항 (Optional)

@cys4585 cys4585 added FE 프론트엔드 관련 이슈입니다. ⚒️ 리팩터링 refactor (기능이 변경되지는 않지만 코드를 수정) labels Oct 20, 2024
@cys4585 cys4585 self-assigned this Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 관련 이슈입니다. ⚒️ 리팩터링 refactor (기능이 변경되지는 않지만 코드를 수정)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant