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

[ Fix ] 온보딩 QA #348

Merged
merged 2 commits into from
Nov 11, 2024
Merged

[ Fix ] 온보딩 QA #348

merged 2 commits into from
Nov 11, 2024

Conversation

yarimu
Copy link
Contributor

@yarimu yarimu commented Nov 11, 2024

#️⃣ Related Issue

Closes #347

✅ Done Task

  • 약관동의 클릭하고 오면 풀림 해결
  • 스크롤 영역 나오는거 해결

💎 PR Point

  • 약관 동의
    기존 약관동의하는 onClick함수가 약관 상세보기를 포함한 태그 ItemWrapper에 달려있어 약관동의 상세를 누르면 onClick함수가 한번 더 적용되어 약관이 풀렸던 것이었습니다! 바로 하위의 ItemLeftWrapper 에 다는 것으로 변경하여 약관 풀림 현상 해결했습니다.
    하지만 이렇게 되면 체크표시부터 글자 영역을 클릭할때만 동의가 되는데 (오른쪽 여백 클릭시 무반응, 약관상세 클릭시 약관이동) 이것에 대해 어떻게 생각하시는지 궁금합니다

  • 스크롤 영역 확인
    학과 렌더링 되는 부분 Container상위의 Wrapper 높이 속성 제거하여 해결했습니다!

📸 Screenshot

  • 약관동의 해결
default.mov
  • 학과 외부 스크롤 해
default.mov

@yarimu yarimu added 🛠 Fix 기존의 버그 수정 예림 labels Nov 11, 2024
@yarimu yarimu requested review from j-nary and se0jinYoon November 11, 2024 09:03
@yarimu yarimu self-assigned this Nov 11, 2024
@se0jinYoon se0jinYoon requested a review from lydiacho November 11, 2024 13:14
Copy link
Collaborator

@se0jinYoon se0jinYoon left a comment

Choose a reason for hiding this comment

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

lgtm!

피알 확인했는데, 대부분 모바일로 접근한다는 점, 그리고 옆에 체크 표시가 있어서 유저가 클릭할 영역이 제한되어 유도되었다는 걸 생각해보면 지금처럼 진행해도 괜찮다고 생각합니다 -!

Copy link
Collaborator

@lydiacho lydiacho left a comment

Choose a reason for hiding this comment

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

요정도 클릭영역 축소는 충분히 괜찮을 것 같습니다 !_!
작동되는 것도 다 확인했습니다 ~ 수고하쎳습니두 !!

@yarimu yarimu merged commit edb956d into develop Nov 11, 2024
1 check passed
@yarimu yarimu deleted the fix/#347/onboardingQA branch November 11, 2024 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠 Fix 기존의 버그 수정 size/xs 예림
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ Fix ] 온보딩 QA
3 participants