Skip to content

Conversation

@S-Gihun
Copy link

@S-Gihun S-Gihun commented May 22, 2025

✏️ 작업 내용

#️⃣ 연관된 이슈

#45


📷 작업 결과

작업 결과 사진을 업로드해주세요.

2025-05-22.3.52.32.mov
2025-05-22.3.52.03.mov

💡 함께 공유하고 싶은 부분

해당 주차를 공부하면서 함께 이야기하고 싶은 주제를 남겨주세요.

(어려웠던 부분과 해결 과정, 핵심 코드, 참고한 자료 등)

window.ScrollY 를 못 읽어오는 현상 Notion에 트러블 슈팅 작성 해두었습니다.


🤔 질문

해당 주차 워크북을 공부하면서 궁금했던 질문들을 남겨주세요.


✅ 워크북 체크리스트

  • 모든 핵심 키워드 정리를 마쳤나요?
  • 핵심 키워드에 대해 완벽히 이해하였나요?
  • 실습/미션을 수행하였나요?

✅ 컨벤션 체크리스트

  • pr 제목을 컨벤션에 맞게 작성하였나요?
  • pr에 해당되는 이슈를 연결하였나요?
  • 적절한 라벨을 설정하였나요?
  • 코드리뷰를 요청하기 위해 reviewer를 등록하였나요?
  • 닉네임/main 브랜치의 최신 상태를 반영하고 있나요?

@S-Gihun S-Gihun self-assigned this May 22, 2025
@S-Gihun S-Gihun added the 💡 Mission 미션 수행 label May 22, 2025
Comment on lines 10 to 14
const throttledSetScroll = useRef(
throttle((y: number) => {
setScrollY(y);
}, 2000)
).current;
Copy link
Contributor

@duwlsssss duwlsssss May 23, 2025

Choose a reason for hiding this comment

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

usethrottle 훅에서 함수를 매개변수로 받게 수정하면 라이브러리 안 써도 될 것 같아요!

Copy link
Author

Choose a reason for hiding this comment

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

useFnThrottle.ts 하나 만들어서 적용해봤습니다 ! 감사합니다.

Copy link

@leetaesk leetaesk left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 !

@S-Gihun S-Gihun merged commit 408431f into jett/main May 23, 2025
@S-Gihun S-Gihun deleted the jett/#45 branch June 20, 2025 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💡 Mission 미션 수행

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants