-
Notifications
You must be signed in to change notification settings - Fork 4
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
✨ Feature - Header #39
Conversation
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.
수고많으셨습니다!!
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.
고생하셨습니다!
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.
수고하셨습니다!!
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.
수고하셨습니다~!
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.
Nav 컴포넌트 파일 잊지말고 리펙토링 한번 다음번에 해주시면 될듯합니다!
수고하셨어요!
(아마 제가 리쿠르팅 페이지 프리페치 박아둬서 충돌날 수 있는데 그 부분만 나중에 머지할때 충돌 해결하심될듯!)
@@ -13,7 +13,8 @@ | |||
"next": "14.1.3", | |||
"react": "^18", | |||
"react-dom": "^18.2.0", | |||
"react-icons": "^5.0.1" | |||
"react-icons": "^5.0.1", |
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.
다음번엔 Tailwind 아이콘으로 써드파트 추가해보는건 어떨지 ㅎㅅㅎ
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.
정말좋습니다
export default function Navigation() { | ||
const pathname = usePathname(); // 현재 경로를 가져오기 | ||
|
||
const [isProjectOpen, setIsProjectOpen] = useState(false); |
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.
이 파일 리펙토링 한번 필요할듯! 너무 무거움!
유틸 따로 뺄수있을듯합니다
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.
ok!
제가 충돌 수정했음 |
PR 타입 ( 하나 이상의 PR 타입을 선택해주세요 )
개요
해더, 모바일 스와이퍼 구현
변경 사항
코드 리뷰시 참고 사항
이미지의 경우 추후 작업해서 마무리 하겠습니다.
stories 부분 스와이퍼가 두개 뜨고 있는데 이것도 1차 피드백 이후 애니메이션 형태에 맞추어 수정하며 수정하겠습니다