Skip to content

[Feature] - 여행기 및 여행 계획 상세페이지 접근성 개선 #566

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

Merged
merged 21 commits into from
Oct 23, 2024

Conversation

simorimi
Copy link
Contributor

✅ 작업 내용

  • 여행기 상세페이지 접근성 개선
  • 여행 계획 상세페이지 접근성 개선

📸 스크린샷

  • 여행기 상세페이지 접근성 점수
    image
2024-10-23.12PM.mov
  • 여행 계획 상세페이지 접근성 점수
    image
2024-10-23.12.36PM.mov

🙈 참고 사항

작업 완료했습니다
체크박스 label 부분이 약간 아쉬운 거 같기도 해서 얘를 개선해볼까싶기도 합니다! 일단 접근성 둘다 90점 넘어서 괜찮은 거 같습니다

@simorimi simorimi added this to the sprint 6 milestone Oct 22, 2024
@simorimi simorimi self-assigned this Oct 22, 2024
Copy link

github-actions bot commented Oct 22, 2024

Test Results

11 tests   11 ✅  14s ⏱️
 2 suites   0 💤
 1 files     0 ❌

Results for commit e0fc8b1.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@0jenn0 0jenn0 left a comment

Choose a reason for hiding this comment

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

시모 고생하셨습니다 👍

const handleCloseDeleteModal = () => {
handleCloseMoreDropdown();
setIsDeleteModalOpen(false);
setDeleteModalAnnouncement("삭제 창이 닫쳤습니다.");
Copy link
Contributor

Choose a reason for hiding this comment

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

창 -> 메뉴로 통일하면 좋을 것 같습니다~!

Copy link
Contributor

@jinyoung234 jinyoung234 left a comment

Choose a reason for hiding this comment

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

작업 고생 많으셨습니다 시모 :)

@simorimi simorimi merged commit f6c4453 into develop/fe Oct 23, 2024
3 checks passed
@simorimi simorimi deleted the feature/fe/#550 branch November 21, 2024 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants