-
Notifications
You must be signed in to change notification settings - Fork 6
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
Design: 마이페이지 설정으로 이름 변경 및 font, theme ver3.0 적용 #266
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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~✨🥹💕
+. PR 머지하면 해당 브랜치는 삭제해야 될 것 같아요..! 👀
src/components/Header/Header.tsx
Outdated
<h1 className={`${styles.headerTitle} ${styles.flexChild}`}>{title}</h1> | ||
|
||
<div className={`${styles.flexChild} ${styles.rightChild}`}>{right}</div> | ||
{right === null ? <></> : <div className={`${styles.flexChild} ${styles.rightChild}`}>{right}</div>} | ||
</div> | ||
); |
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.
서영님, right이 null일때 Fragment를 추가해 둔 이유가 궁금합니다~!
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.
소현님 저도 제가 왜그랬는지 궁금합니다..! 이 점 반영하여 리팩토링 하였고, 페이지 디자인 다시 하여 수정하였습니다~!
그리고 git push도 upstream으로 곧바로 해버렸네요ㅠㅠ 이번에는 머지 후 지우고, 다음부터는 origin push 후에 PR 올리겠습니다~!
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.
서영님 새로 반영한 디자인 너무 보기 좋아요!! 👍 또, 드롭다운이 아닌 토글모양 선택도 직관적으로 옵션을 보여주는 것 같아요✨ 역시 최고입니다!!🥹
개요
작업 사항
참고 사항 (optional)
관련 이슈 (optional)
스크린샷
⬇︎ 다시 디자인 한 버전
리뷰어에게