-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add interviews header #46
Conversation
kwonmory
commented
Aug 28, 2020
•
edited
edited
- add interviews header
- design interviews header
- change NotFoundPage styled style to object
- fix NotFoundPage function name...
- change NotFoundPage styled style to object - fix NotFoundPage function name...
오 근데 저 햄스터 이미지 사용해도 되는 이미지겠죠? ㅎㅎ; |
저 햄스터(애비츄)는 어떤 분께서 블로그에 파워포인트로 템플릿 공유하신거 부분만 따와서 사용했는데.. |
<Wrapper> | ||
<div> | ||
<TitleStyled> | ||
{quiz?.title || '인터뷰즈_'} |
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 renderQuizHeader({ PATH, button = null, quiz = null }) { | ||
return render( | ||
<MemoryRouter> | ||
<QuizHeader path={PATH} onExit={button} quiz={quiz} /> | ||
</MemoryRouter>, | ||
); | ||
} |
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.
음.. null을 허용한다는게 조금 마음이 걸리네요. null을 허용하는 순간 null인지 아닌지 항상 검사해야되서 복잡성이 증가하거든요
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.
아... 그렇군요. 전 또 그냥 값이 없으면 null로 초기화라고 해야지 하고 넣었는데, 원래 값의 기본값으로 초기화 처리 해야겠어요.