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: 스타일 svg 폰트 설정 #5

Merged
merged 6 commits into from
Jan 13, 2025
Merged

feat: 스타일 svg 폰트 설정 #5

merged 6 commits into from
Jan 13, 2025

Conversation

ptq124
Copy link
Collaborator

@ptq124 ptq124 commented Jan 12, 2025

이슈 번호

작업한 목록을 작성해 주세요

  • svg
  • 폰트, 컬러
  • theme.css.ts. 토큰 구성

스크린샷

pr 포인트나 궁금한 점을 작성해 주세요

  • ui 내부 폴더 구성 어떻게 할까요?
  • 피그마에 있는 변수 이름 최대한 사용
  • 컬러 폰트 className으로 적용하면 됨

연관된 issue: #4

@DongjaJ
Copy link
Collaborator

DongjaJ commented Jan 13, 2025

ui 내부에서 컴포넌트 별로 디렉토리 만들어서 만들면 좋을 것 같아요!
피그마에 있는 변수 이름대로 하신 것 좋습니다.
근데 나중에 변수 이름 바꿔달라고 디자이너분께 요청해도 좋을 것 같네여.
고생하셨습니다!

Copy link
Collaborator

Choose a reason for hiding this comment

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

svg 이름이 이러면 import를 어떻게 해야하나요?

import IcLiverLarge from 'name=ic_ liver, size=large.svg'

이렇게 해야하는 거면 from 뒤에 문자열을 입력하는데 불편할 것 같아요.
파일 이름을 깔끔하게 IcLiverLarge 처럼 수정 하는게 좋을 것 같은데 태구님은 어떻게 생각하시나요?

Copy link
Collaborator

@DongjaJ DongjaJ Jan 13, 2025

Choose a reason for hiding this comment

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

그리고 저는 svg를 사용할때 아래처럼 사용하는 걸 선호하는데 태구님은 svg파일 그대로 사용하는 걸 선호하시나여?
아래처럼 쓰면 svgr도 필요 없을 것 같아요. 이건 요청드리는건 아니고 태규님 의견이 궁금합니다!

export default function SvgIcon(){ return <svg> ... </svg> }

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

  1. assets/index.ts 파일에 미리 등록을 해놔서 자동완성 되므로 from 문자열은 직접 입력할 일은 없을 거 같아요! 시각적으로 불편하시다면 바꾸는 거 가능합니당!ㅋㅋㅋㅋㅎㅋ
  2. 처음 작업할때 아이콘이 많아서 피그마에 통째로 일일이 컴포넌트로 변환하기 번거로워 svgr 방식으로 만들었습니다. (컴포넌트 작업은 시간이 더 필요함) svgr도 svg를 react 컴포넌트로 변환하는 거라 동적 변환에는 문제가 없는데 개인차가 있는 거 같아요. 저는 어떤 형식이든 상관 없습니다!

Copy link
Collaborator

Choose a reason for hiding this comment

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

아하 제가 index.ts를 확인못했네요. 이해했습니다~

Copy link
Collaborator

Choose a reason for hiding this comment

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

font는 지금은 이거로 하고 나중에 woff2로 교체해요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

검색해보니 otf가 용량이 꽤 크네요... woff2가 있는지 몰랐습니당. 새로운 걸 배웠네요
넵 일단 폰트는 나중에 교체 하는거로 해용!

@ptq124 ptq124 merged commit 9d7e0f4 into main Jan 13, 2025
7 checks passed
@ptq124 ptq124 deleted the feat-4 branch January 13, 2025 14:07
@ptq124
Copy link
Collaborator Author

ptq124 commented Jan 13, 2025

추가로 디자이너 분이 컴포넌트 page에 이름 지은 네이밍 그대로 가려 했는데 실제 view에 있는 이름과 차이가 있는 거 같아요...
디자이너와 개발자간에 네이밍 회의는 해야될거 같습니다.

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.

feat: 스타일 svg 폰트 설정
2 participants