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

Merged
merged 5 commits into from
Oct 23, 2024
Merged

웹 접근성 개선 #693

merged 5 commits into from
Oct 23, 2024

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 requested review from ss0526100 and jaeml06 October 20, 2024 09:24
@cys4585 cys4585 self-assigned this Oct 20, 2024
Copy link
Contributor

@ss0526100 ss0526100 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~~

const handleResize = () => {
if (window.visualViewport) {
const keyboardHeight =
window.innerHeight - window.visualViewport.height;
setKeyboardHeight(Math.max(0, keyboardHeight));
console.log(Math.max(0, keyboardHeight));
Copy link
Contributor

Choose a reason for hiding this comment

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

console.log 지워줘요잉~~~

@jaeml06 jaeml06 merged commit 0e2b6d9 into develop-frontend Oct 23, 2024
1 check failed
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.

3 participants