-
Notifications
You must be signed in to change notification settings - Fork 2
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
[FE][Feat] #112 : 레이아웃 상단 헤더 구현 및 공통 레이아웃 적용 (라우터 수정) #168
Conversation
TSDoc이나, 스토리북, 테스트에 대해서는 우선 빠르게 개발을 마치고 주말중에 작업해두겠습니다. 변경사항이 너무 잦아서 데모가 내일인 관계로 전체적인 구조가 나오면 그때부터 적고자 합니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확실히 훨씬 더 깔끔해지고 범용적으로 바뀐 것 같아 좋아요!! 제가 막 짠 코드 수정하시느라 고생 많으셨습니다...ㅎㅎ
- useContext와 useMemo, useState 조합해서 처리할 수 있도록 구현
다시한번 리뷰 부탁드립니다. 재수정해서 올렸습니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
context 활용해서 구현하셨군요!! 참고해보도록 하겠습니다! 고생하셨습니다
📝 PR 개요
🔍 변경 사항
useContext
이용해서 수정✅ 체크리스트 (Checklist)
🔄 관련 이슈 (Linked Issues)
#112
#110
#111
#112
#109
#134
📷 스크린샷 및 동영상 (선택 사항)
2024-11-14.4.04.19-1.mov
⚙️ 기타 사항