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] 스크린 리더기 focus 개선 #347

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

Conversation

novice0840
Copy link
Contributor

@novice0840 novice0840 commented Oct 17, 2024

Issue Number

#335

As-Is

    1. iOS 스크린 리더기에서 페이지 이동 시 focus가 Header로 이동하지 않는 상황
    1. 스크린 리더기에서 Modal이 뜰 경우 Modal이 활성화된 것을 인지하지 못하고 focus가
      Modal 뒤에서만 이동함
    1. 스크린 리더기에서 Modal이 닫힐 경우 원래의 자리를 찾지 못함

To-Be

    1. 페이지가 처음 로드될 때 Header로 focus를 옮김
header-focus.mp4
    1. Modal이 활성화 될 때 Modal로 focus를 옮김
modal-focus.mp4
    1. Modal이 닫힐 때 Modal로 focus를 원래대로 돌림
modal-close.mp4

Check List

  • 테스트가 전부 통과되었나요?
  • 모든 commit이 push 되었나요?
  • merge할 branch를 확인했나요?
  • Assignee를 지정했나요?
  • Label을 지정했나요?

(Optional) Additional Description

scroll을 막는 hook이 있었는데 서비스에서 애초에 scroll을 막기 때문에 사용하지 않으므로 제거함

저번 PR에서 소리가 잘 들리지 않는 다는 피드백이 있어 이번에는 리더기 볼륨을 최대로 키웠습니다.

🌸 Storybook 배포 주소

https://woowacourse-teams.github.io/2024-ddangkong/storybook/

🌸 Storybook 배포 주소

https://woowacourse-teams.github.io/2024-ddangkong/storybook/

@novice0840 novice0840 added ✨ feat 기능 추가 🫧 FE front end labels Oct 17, 2024
@novice0840 novice0840 added this to the FE Sprint6 milestone Oct 17, 2024
@novice0840 novice0840 self-assigned this Oct 17, 2024
@novice0840 novice0840 linked an issue Oct 17, 2024 that may be closed by this pull request
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🫧 FE front end ✨ feat 기능 추가
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

[FEAT] 스크린 리더기 focus 개선
1 participant