-
Notifications
You must be signed in to change notification settings - Fork 3
Feat/#185 moodpiickle design #186
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
Conversation
joohaem
left a comment
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/style/theme.ts
Outdated
| body1: FONT({ weight: 600, size: 2.4, lineHeight: 140, letterSpacing: 3 }), | ||
| body2: FONT({ weight: 600, size: 2.0, lineHeight: 140, letterSpacing: 0 }), | ||
| body3: FONT({ weight: 500, size: 1.8, lineHeight: 140, letterSpacing: 35 }), | ||
| body3: FONT({ weight: 500, size: 1.8, lineHeight: 140, letterSpacing: 3.5 }), |
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.
🚨
이거 3.5 아니라 35로 넣어줘야 해요!
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: -0.0${letterSpacing}rem;
이렇게 되어있어서 -0.03.5rem 이 되어버립니다
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.
더불어서, 배너 이미지가 width 720px 인데, => 보류
우리 서비스가 PC에서도 360px 을 안넘어서 이미지 줄여달라고 제가 요청할게요!
유니코드랑 디자인/서버와 소통하면서 작업하시느라 고생 많으셨습니다😭😭😭
📌 내용
📌 내가 알게 된 부분
👀 이모지 생성 관련 배운 지식 👀
String.fromCodePoint(parseInt(item.unicode, 16))이렇게 16진수 코드만 넣어주면 그에 맞는 문자열을 반환해주는 메서드가 있다는 점을 알게 되었습니다.&#x를 붙이면 16진수,&#만 붙이면 10진수!📸 스크린샷