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

intersection observer문제 수정 #31

Merged
merged 2 commits into from
Sep 10, 2024
Merged

intersection observer문제 수정 #31

merged 2 commits into from
Sep 10, 2024

Conversation

dladncks1217
Copy link
Contributor

Why need this PR❓

intersection observer문제 수정

Changes ✌️

intersection observer문제 수정

containerRef.current를 찾는 시점에서는 이미 로딩컴포넌트가 로딩 된 이후인지라, useEffect가 먼저 돌고 ref를 null로 읽어 발생하는 이슈였습니다.
조건 하나 추가해줬어용

@dladncks1217 dladncks1217 self-assigned this Sep 10, 2024
Copy link
Contributor

@moong23 moong23 left a comment

Choose a reason for hiding this comment

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

lgtm 👍

@moong23 moong23 merged commit 748e64e into dev Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants