-
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
Header 컴포넌트 추가 #52
Header 컴포넌트 추가 #52
Conversation
pathname: string | ||
} | ||
|
||
const Hero: React.FC<HeroProps> = ({ image, title, description, onViewportEnter, onViewportLeave, pathname }) => { |
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.
컴포넌트 정의 컨벤션을 function 스타일로 한다고 하셨지 않나용??
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.
스토리보드, 스타일 sub 확장자가 붙어있는 파일들 어떤식으로 사용되는지 알게되었습니다 :)
다만 Hero 컴포넌트 에로우 펑션으로 정의했던데, function 스타일로 하기로 하신거 아니었나요?
0397364
to
efb369c
Compare
efb369c
to
3ea2837
Compare
} | ||
|
||
const title = 'JaeSeoKim' | ||
const description = '🎢 To become a better developer...!' |
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.
저두 이 부분 궁금합니당!! config.json 파일에서 직접가져올 건지 혹은 context API 같은 걸 사용할 건지?? 궁금해요!!
fontSize: '24px', | ||
<LayoutGroup> | ||
<Styled.rootContainer | ||
initial={[isShowHero ? 'hero' : 'header']} |
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.
이 부분의 props들에 대한 설명 월요일에 부탁드립니다! 궁금해유
혹은 어떤 라이브러리 문서 참고하면 되는지 알려주셔도 좋을 것 같습니다
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.
LGTM
Header 컴포넌트 제작