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: 디자인 시스템 색상 및 폰트 적용 #40

Merged
merged 11 commits into from
Dec 14, 2024

Conversation

hyjoong
Copy link
Contributor

@hyjoong hyjoong commented Dec 14, 2024

무엇을 위한 PR인가요?

  • 신규 기능 추가 : 디자인 시스템 색상 및 폰트 수정
  • 버그 수정 :
  • 리펙토링 :
  • 문서 업데이트 :
  • 기타 :

변경사항 및 이유
이유: 새로운 디자인 시스템 가이드라인에 맞춰서 색상 체계 및 폰트 적용이 필요

작업 내역

  • tailwind 설정 변경

    • gray색상 체계 추가
    • NanumDaHaengCe 폰트 설정 추가
  • 컴포넌트 스타일 수정

    • button 컴포넌트
    • input 컴포넌트

PR 특이 사항
새로운 색상 시스템에 gray색상이 많아서 gray-{sale} 으로 변경했습니다.
기존에 border-gray-500 과 같이 하드코딩한 색상이 config에 설정된 색상을 수정돼서 스타일이 꺠질 가능성이 있습니다.
image

  • NanumDaHaengCe 폰트를 적용하려면 다음과 같이 코드를 작성하면 됩니다.
font-nanum-dahaengce

스크린샷

image

Issue Number
close: #
어떤 부분에 리뷰어가 집중하면 좋을까요?

피그마에 적용된 배경색을 제대로 줬는지 잘 모르겠네요.

@hyjoong hyjoong self-assigned this Dec 14, 2024
@hyjoong hyjoong marked this pull request as ready for review December 14, 2024 07:55
Copy link
Member

@s2oy s2oy left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

@Jeong-jj Jeong-jj left a comment

Choose a reason for hiding this comment

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

이전 #39 merging 부분과 특별히 conflict 날 부분은 없을 것으로 보이는데
혹시나 refactoring과정에서 파일 row 개수가 줄어든 경우가 있을 수도 있으니 한번 더 체크 후 re-push 부탁드리겠습니다!

@hyjoong hyjoong requested a review from Jeong-jj December 14, 2024 12:43
@hyjoong
Copy link
Contributor Author

hyjoong commented Dec 14, 2024

@Jeong-jj pull받고 re-push 했습니다 ! 추가로 피그마에 textarea 디자인이 나와서 추가 작업했습니다 a2e59d9

@Jeong-jj
Copy link
Member

@Jeong-jj pull받고 re-push 했습니다 ! 추가로 피그마에 textarea 디자인이 나와서 추가 작업했습니다 a2e59d9

감사합니다!! 고생하셨어요!

@hyjoong hyjoong merged commit 5e9c25a into main Dec 14, 2024
@Jeong-jj Jeong-jj deleted the feature/design-system branch December 18, 2024 07:27
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.

4 participants