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: handy typo 적용 #119

Merged
merged 10 commits into from
Jul 18, 2024
Merged

feat: handy typo 적용 #119

merged 10 commits into from
Jul 18, 2024

Conversation

fecapark
Copy link
Contributor

@fecapark fecapark commented Jul 8, 2024

1️⃣ 어떤 작업을 했나요? (Summary)

  1. Pretendard subset 폰트 로컬에 추가, 불러올 수 있도록 preview-head 수정
  2. style/foundation/typo/typo.type.ts 구조 수정
  3. typo 문서를 더 자세하게 변경

기존 코드에 영향을 미치지 않는 변경사항

기존 코드에 영향을 미치는 변경사항

typo 구조를 수정하면서 타입 또한 변경되었습니다.

  • 단일 Typo 타입 -> TypoKRType , TypoENType 으로 분리.
  • style/theme/theme.type.tsYDSTheme 타입의 typo 필드의 타입이 KRENTypos 로 변경.
// 참고
interface KRENTypos {
  en: Record<TypoENType, string>;
  kr: Record<TypoKRType, string>;
}

버그 픽스

2️⃣ 알아두시면 좋아요!

3️⃣ 추후 작업

4️⃣ 체크리스트 (Checklist)

  • main 브랜치의 최신 코드를 pull 받았나요?

@fecapark fecapark self-assigned this Jul 8, 2024
@fecapark fecapark added the feat label Jul 8, 2024
Copy link
Collaborator

@nijuy nijuy left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 :3

#99 에서 웹폰트 로딩을 위한 작업이 있었는데
폰트 파일을 로컬에 위치시킬 거라면 해당 pr에서 설치했던 라이브러리 삭제 + globalStyle 수정해도 될 거 같어요

그리고 fonts 폴더가 src 상위에 있어서 배포할 때 포함이 되.......나? 싶은데
제가 지금 패드라서 실행을 직접 못해봐가지고,, 일단 코멘트만 달아두겠습니다 !!!

@fecapark
Copy link
Contributor Author

고생하셨습니다 :3

#99 에서 웹폰트 로딩을 위한 작업이 있었는데 폰트 파일을 로컬에 위치시킬 거라면 해당 pr에서 설치했던 라이브러리 삭제 + globalStyle 수정해도 될 거 같어요

그리고 fonts 폴더가 src 상위에 있어서 배포할 때 포함이 되.......나? 싶은데 제가 지금 패드라서 실행을 직접 못해봐가지고,, 일단 코멘트만 달아두겠습니다 !!!

저희 쪽에서 로컬 폰트를 적용해줄 수 있는 방법이 있나요?

당연하게 숨쉴때에서 관리해야한다고 생각했는데 혹시 해결한 방법이 ?!

Copy link
Member

@2wndrhs 2wndrhs left a comment

Choose a reason for hiding this comment

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

수고하셨습니다👍

Copy link
Member

Choose a reason for hiding this comment

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

웹폰트
CDN을 이용해 Pretendard를 사용할 수 있으며, 토글을 확인해 기본적으로 추천하는 jsDelivr 외에도 cdnjs와 UNPKG 중에 원하는 CDN을 사용하실 수 있습니다.
https://github.com/orioncactus/pretendard

웹폰트로도 Pretendard를 사용할 수 있던데 로컬에서 폰트를 서빙하는게 성능상 이점이 있을까요?

@nijuy
Copy link
Collaborator

nijuy commented Jul 10, 2024

저희 쪽에서 로컬 폰트를 적용해줄 수 있는 방법이 있나요?

당연하게 숨쉴때에서 관리해야한다고 생각했는데 혹시 해결한 방법이 ?!

src 하위에 fonts를 넣으면 될 거 같긴 한데 정확한 건 저도 tgz 만들어서 테스트 해봐야 알 거 같네용 ㅎ^^...
핸디를 숨쉴때에서만 쓸 게 아니니 사용처에 폰트 관리를 맡기는 것보단 웹폰트 방식을 유지하는 건 별론가요?!
결국 준 코멘트(로컬에서 서빙하는 이점이 궁금하다) 내용이긴 합니다 ㅎㅎ

@2wndrhs
Copy link
Member

2wndrhs commented Jul 11, 2024

image

폰트가 한글폰트로 통합되었다고 하네요🥲

@fecapark
Copy link
Contributor Author

@nijuy @2wndrhs

웹폰트 있으면 무조건 써야죠 !!!

SSR 환경만 아니라면 대환영입니다

웹폰트로 변경하겠습니다

@nijuy
Copy link
Collaborator

nijuy commented Jul 18, 2024

이거 어제 새벽까진 완료된다고 생각하고 있었는데 혹시 아닌가요?? (⊙o⊙)

@fecapark
Copy link
Contributor Author

이거 어제 새벽까진 완료된다고 생각하고 있었는데 혹시 아닌가요?? (⊙o⊙)

어제 새벽에 완료했습니다

커밋

Copy link
Collaborator

@nijuy nijuy left a comment

Choose a reason for hiding this comment

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

웹폰트 수정 커밋만 보고 아직 안된둘 알았어요 죄송 ^^,,,,, 고생하셨습니다~~~!

@fecapark fecapark merged commit 6c0f801 into develop Jul 18, 2024
@fecapark fecapark deleted the feat/#118-handy-typo branch July 18, 2024 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: Handy 타이포 시스템 적용
3 participants