-
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
Feat/#78 nayeon qa #82
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.
고생했어요 !!!@~ 아래 리뷰들만 수정하고 머지할게요 !!
src/components/Main/Banner/style.ts
Outdated
letter-spacing: -0.03em; | ||
${({ theme }) => theme.fonts.caption4}; | ||
color: ${({ theme }) => theme.colors.white}; |
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.
letter-spacing
은 theme 에 있는 속성인데 이 글자만 달랐던 걸까요??
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.
-0.04 로 설정되어 있어서 그렇구나- 제가 수정할게요!
|
||
letter-spacing: -0.03em; | ||
${({ theme }) => theme.fonts.body3}; | ||
color: ${({ theme }) => theme.colors.gray800}; |
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.
letter-spacing 은 theme 에 있는 속성인데 이 글자만 달랐던 걸까요??
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.
나연찡 너무너무 잘했당 ㅜㅜㅜ 어제 엄청 열심히 작업했구나!!!!!! 멉쪙!!!!!!!!!!💙💙💙
// const location = useLocation(); | ||
// const CARD_TYPE = location.state as number...?; | ||
// eslint-disable-next-line prefer-const | ||
let CARD_TYPE_LOCATION: CardsTypeLocation = { type: "category", categoryId: "62cbb7d8a8c54f168a6ddfe1" }; |
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.
안쓰는 변수 지워주세요~~~(이주함 따라한거임)
@@ -8,7 +8,7 @@ export const St = { | |||
Form: styled.form``, | |||
|
|||
Title: styled.h2` | |||
margin-bottom: 2rem; | |||
padding: 2.4rem 0; |
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.
요게 채민이가 말한 제목 사이 패딩값이애오!??! Header에 일정하게 margin 주고 페이지마다 일일히 -margin 값 주려고 했는데 요게 맞나....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.
엇 언니가 하려던대로 하려면 수정해야 합니당!!
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.
Good~~!!!!
|
||
letter-spacing: -0.03rem; | ||
${({ theme }) => theme.fonts.body4}; | ||
${({ theme }) => theme.fonts.body6}; |
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.
font는 따로 지정을 안해주나용 ?
📌 내용
📌 내가 알게 된 부분
📌 질문할 부분
📸 스크린샷
밑의 사진은 그냥 넣어봤어요~~