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

[ J039 ] : FE => Profile Modal Close Event #133

Merged
merged 3 commits into from
Nov 11, 2021
Merged

Conversation

jin-Pro
Copy link
Member

@jin-Pro jin-Pro commented Nov 10, 2021

Profile Modal close CustomHook 관련해서
closest를 사용해서 고정된 className을 찾아 target 포함 여부로 모달상태를 조절하였다.

closest를 사용하면 dom에 접근하게되어 DOm 접근을 줄이기위해서
useRef를 배열로 사용하여 로직을 수정해서 관리해주었다.

@jin-Pro jin-Pro added this to the 프로젝트 3주차 milestone Nov 10, 2021
@jin-Pro jin-Pro added the 👑 리뷰 요청 리뷰 요청 label Nov 10, 2021
@jin-Pro jin-Pro added this to 완료 in 3주차 Nov 10, 2021
@jin-Pro jin-Pro merged commit ad6c4d3 into dev Nov 11, 2021
@jin-Pro jin-Pro deleted the Feat/ModalCustomHook branch August 8, 2022 01:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👑 리뷰 요청 리뷰 요청
Projects
3주차
완료
Development

Successfully merging this pull request may close these issues.

None yet

1 participant