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

[유저] 회원가입 페이지 성별 선택 불가 이슈 해결 #610

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

Gwak-Seungju
Copy link
Contributor

What is this PR? 🔍

Changes 📝

  • 성별 선택 드롭다운에 onBlur 이벤트를 추가하여 바깥 클릭 시 드롭다운이 닫히도록 구현했었습니다. 그 상태로 성별 선택을 위해 li 아이템에 onClick 이벤트를 실행하려 하니 성별 선택이 되지 않은 채로 드롭다운 메뉴가 닫히는 현상이 발생했습니다. 그 이유가 이벤트 핸들러의 작동 순서 때문이라는 것을 알게되었습니다.
    onBlur -> onClick 순으로 이벤트 핸들러가 작동하기 때문에 onClick 이벤트가 실행되지 않고 드롭다운이 닫히는 것이었습니다! 따라서 onBlur의 작동 순서보다 앞서는 onMouseDown 이벤트를 사용해 해결했습니다.

✔️ Please check if the PR fulfills these requirements

  • It's submitted to the correct branch, not the develop branch unconditionally?
  • If on a hotfix branch, ensure it targets main?
  • There are no warning message when you run yarn lint

@Gwak-Seungju Gwak-Seungju added 🐞 BugFix 버그 수정 👤 User 유저, 시간표 도메인 labels Dec 19, 2024
@Gwak-Seungju Gwak-Seungju self-assigned this Dec 19, 2024
Copy link
Contributor

@Yejin0070 Yejin0070 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@hyejun0228
Copy link
Contributor

수고하셨습니다~!

@Gwak-Seungju Gwak-Seungju merged commit 55e9842 into develop Dec 19, 2024
3 checks passed
@github-actions github-actions bot deleted the fix/#609/gender-select-issue branch December 19, 2024 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 BugFix 버그 수정 👤 User 유저, 시간표 도메인
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[유저] 회원가입 시 성별 선택 안 되는 이슈
3 participants