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

Header 컴포넌트 추가 #52

Merged
merged 4 commits into from
Apr 25, 2022
Merged

Header 컴포넌트 추가 #52

merged 4 commits into from
Apr 25, 2022

Conversation

JaeSeoKim
Copy link
Contributor

Header 컴포넌트 제작

@JaeSeoKim JaeSeoKim self-assigned this Apr 21, 2022
@JaeSeoKim JaeSeoKim linked an issue Apr 21, 2022 that may be closed by this pull request
pathname: string
}

const Hero: React.FC<HeroProps> = ({ image, title, description, onViewportEnter, onViewportLeave, pathname }) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

컴포넌트 정의 컨벤션을 function 스타일로 한다고 하셨지 않나용??

Copy link
Contributor

@Likilee Likilee left a comment

Choose a reason for hiding this comment

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

스토리보드, 스타일 sub 확장자가 붙어있는 파일들 어떤식으로 사용되는지 알게되었습니다 :)
다만 Hero 컴포넌트 에로우 펑션으로 정의했던데, function 스타일로 하기로 하신거 아니었나요?

@JaeSeoKim JaeSeoKim force-pushed the feat/blog/header branch 2 times, most recently from 0397364 to efb369c Compare April 23, 2022 08:03
}

const title = 'JaeSeoKim'
const description = '🎢 To become a better developer...!'
Copy link
Contributor

Choose a reason for hiding this comment

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

이부분 어떻게 유저정보에서 받아오도록 할건지 궁금해요!

Copy link
Member

@nyeoni nyeoni Apr 24, 2022

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']}
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분의 props들에 대한 설명 월요일에 부탁드립니다! 궁금해유
혹은 어떤 라이브러리 문서 참고하면 되는지 알려주셔도 좋을 것 같습니다

Copy link
Contributor

@Yenowme Yenowme left a comment

Choose a reason for hiding this comment

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

LGTM

@JaeSeoKim JaeSeoKim merged commit 93d67bf into main Apr 25, 2022
@JaeSeoKim JaeSeoKim deleted the feat/blog/header branch April 25, 2022 13:02
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.

헤더 컴포넌트
4 participants