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

feat: 색상, 타이포 정리 #134

Merged
merged 8 commits into from
Jul 19, 2024
Merged

feat: 색상, 타이포 정리 #134

merged 8 commits into from
Jul 19, 2024

Conversation

hsgh085
Copy link
Collaborator

@hsgh085 hsgh085 commented Jul 17, 2024

💡 작업 내용

  • 색상 정리
  • 타이포 정리

💡 자세한 설명

디자이너님들이 정의해주신 색상 팔레트와 타이포를 변수명 그대로 정리했습니다!

추가적으로 hook과 api 함수를 리팩토링하면서 프로젝트를 실행하니 수많은 에러가 터지는 것을 확인했습니다,,!
그래서 랜딩페이지가 정상적으로 랜딩되기 위해서 임시방편으로 Router들을 주석 처리 했습니다🥲

이전 색상 및 타이포 타입을 삭제하니 스토리북 배포에 실패해서 삭제하지 않고 다시 놔두었어요ㅠ 해당 색상 및 타이포를 참조하는 코드들이 리팩토링 된 이후에 삭제하도록 할께요!

📗 참고 자료 (선택)

📢 리뷰 요구 사항 (선택)

🚩 후속 작업 (선택)

✅ 셀프 체크리스트

  • PR 제목을 형식에 맞게 작성했나요?
  • 브랜치 전략에 맞는 브랜치에 PR을 올리고 있나요? (master/main이 아닙니다.)
  • 이슈는 close 했나요?
  • Reviewers, Labels, Projects를 등록했나요?
  • 작업 도중 문서 수정이 필요한 경우 잘 수정했나요?
  • 테스트는 잘 통과했나요?
  • 불필요한 코드는 제거했나요?

closes #133

@hsgh085 hsgh085 added ✔︎pull requests pull requests 코드 체크 요청 👩🏻‍💻 frontend 프론트엔드 작업 ✅feature labels Jul 17, 2024
@hsgh085 hsgh085 requested a review from areumH July 17, 2024 16:53
@hsgh085 hsgh085 self-assigned this Jul 17, 2024
Copy link
Contributor

@areumH areumH left a comment

Choose a reason for hiding this comment

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

피그마 타이포를 확인해보니 숫자는 poppins라는 폰트로 따로 있는 것 같아요!!
폰트 다운로드 관련하여 디자이너분들께 여쭤보거나 구글 폰트에서 다운받거나 해야할 것 같아용

@hsgh085
Copy link
Collaborator Author

hsgh085 commented Jul 17, 2024

헉 그렇네요,,! 숫자 폰트는 다른 폰트였군요,,! 디자이너님들이랑 회의할 때 한 번 폰트 관련해서 여쭤보고 수정할께요!

@areumH areumH self-requested a review July 18, 2024 15:55
Copy link
Contributor

@areumH areumH left a comment

Choose a reason for hiding this comment

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

스토리북 폰트 적용을 위해 .storybook/preview-head.html 파일에 Poppins 폰트 추가 후 머지하시면 될 것 같아요!!
수고 많으셨습니다 🙌

@hsgh085 hsgh085 merged commit 6a3f42e into dev Jul 19, 2024
3 checks passed
@hsgh085 hsgh085 deleted the feat/133-design-system branch September 27, 2024 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✅feature 👩🏻‍💻 frontend 프론트엔드 작업 ✔︎pull requests pull requests 코드 체크 요청
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

색상 팔레트, 타이포 타입 정리
2 participants