Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@NYeonK
Copy link
Contributor

@NYeonK NYeonK commented Jan 21, 2023

  • 브랜치명, 브랜치 알맞게 설정
  • Reviewer, Assignees, Label, Milestone, Issue(PR 작성 후에) 붙이기
  • PR이 승인된 경우 해당 브랜치는 삭제하기

📌 내용

  • MoodPiickle 페이지의 배너 수정 및 이모지를 추가했습니다!
  • 그라데이션 배경 추가 완료!

📌 내가 알게 된 부분

👀 이모지 생성 관련 배운 지식 👀

  1. String.fromCodePoint(parseInt(item.unicode, 16)) 이렇게 16진수 코드만 넣어주면 그에 맞는 문자열을 반환해주는 메서드가 있다는 점을 알게 되었습니다.
  2. 유니코드는 앞에 &#x를 붙이면 16진수, &#만 붙이면 10진수!
  3. 유니코드 형태로 디비에서 불러오면 적용이 안됩니다,,,,,,,,,,,,,,,,,진짜 왜지 이것저것 다 해봤는데 ㄹㅇ 안됨

📸 스크린샷

@NYeonK NYeonK requested a review from joohaem January 21, 2023 20:11
@NYeonK NYeonK linked an issue Jan 21, 2023 that may be closed by this pull request
6 tasks
@NYeonK NYeonK self-assigned this Jan 21, 2023
@NYeonK NYeonK marked this pull request as ready for review January 22, 2023 09:44
Copy link
Member

@joohaem joohaem left a comment

Choose a reason for hiding this comment

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

고생해쒀요...!!!!!
나중에 리뷰 남긴 거랑 그라데이션만 추가해서 머지합시다!

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 }),
Copy link
Member

Choose a reason for hiding this comment

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

🚨
이거 3.5 아니라 35로 넣어줘야 해요!

Copy link
Member

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 이 되어버립니다

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아 구렇구나,,! 잘못 표기된줄 알고 수정했오😅😅 바로 수정할게요오

Copy link
Member

@joohaem joohaem left a 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 을 안넘어서 이미지 줄여달라고 제가 요청할게요!
=> 보류

유니코드랑 디자인/서버와 소통하면서 작업하시느라 고생 많으셨습니다😭😭😭

@joohaem joohaem merged commit 67274c3 into develop Jan 23, 2023
@joohaem joohaem deleted the feat/#185-moodpiickle_design branch January 23, 2023 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ MoodPiickle ] 카테고리뷰 디자인 변경

3 participants