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

[#34] Footer 작업 #37

Merged
merged 11 commits into from
Aug 14, 2023
Merged

[#34] Footer 작업 #37

merged 11 commits into from
Aug 14, 2023

Conversation

seocylucky
Copy link
Member

Summary

image

Description

image
  • 관련 문서:

Details

  • 푸터 작업

To Reviewer

  • 사진 비교해보니까 실제 스타일이랑 피그마 스타일이랑 다르네여..? 폰트 크기 스타일대로 적용시킨거긴 한데 흠 일단 확인 한 번 해주세욥!

After Work

  • 이용약관, 개인정보처리방침, 운영정책은 노션 링크로 연결할 예정

Reference

@seocylucky seocylucky requested a review from intersoom August 14, 2023 07:42
@seocylucky seocylucky self-assigned this Aug 14, 2023
Copy link
Member

@intersoom intersoom left a comment

Choose a reason for hiding this comment

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

수고하셨어용~!

gap: 30px;
`;

const WHiteText = styled.div<TextProps>`
Copy link
Member

Choose a reason for hiding this comment

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

H까지 대문자로 한 이유가 있나요? 정말 궁금해서 ㅋㅋㅋㅋ

Copy link
Member Author

Choose a reason for hiding this comment

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

엌ㅋㅋㅋㅋㅋㅋㅋ오타ㅎㅎㅎ

const WHiteText = styled.div<TextProps>`
color: ${theme.colors.white};
font-size: ${(props) =>
props.size === 'M' ? theme.fontStyles.Text_M.fontSize : theme.fontStyles.Text_L.fontSize}px;
Copy link
Member

Choose a reason for hiding this comment

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

이거 true일 경우, px이 안 들어가있어서 그런거 아닐까용?

Copy link
Member Author

Choose a reason for hiding this comment

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

{}px 한꺼번에 묶어서 한건데.. 후우움

Copy link
Member

Choose a reason for hiding this comment

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

엣 그러네요,,

@seocylucky seocylucky merged commit 763c967 into dev Aug 14, 2023
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.

2 participants