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

공유 이미지 생성 시 한자 글리프가 깨지는 문제 고치기 #2025

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mu-hun
Copy link
Contributor

@mu-hun mu-hun commented Apr 29, 2024

출력 선택한 문장
image

재현해보기: ベクターフィッシュ | 글리프

시스템 글꼴 sans-serif, serif 를 fallback 으로 설정했습니다.

얼마 전 퇴사 후 갖고 있는 Doppler 및 Tailscale 계정이 없어 PR Preview 를 통해 검수하고자 우선 Draft PR 로 올려둡니다.


PS. satori 를 같이 사용 중인 opengraph 이미지 생성 기능에서는 정한 폰트 패밀리에서 지원하고 있어 다행히 해당 문제가 없음을 확인했습니다:

https://github.com/withglyph/glyph/blob/ee703747466ac98a023483c63c5928b1192dfcef/apps/website/src/lib/server/rest/routes/opengraph.ts#L185

https://withglyph.com/yale/1419574796

@mu-hun mu-hun added the preview 👀👀👀 label Apr 29, 2024
Copy link

graphite-app bot commented Apr 29, 2024

Your org requires the Graphite merge queue for merging into main

Add the label “:rocket:” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

@robin-maki
Copy link
Member

k8s 팟 안에서는 기본 시스템 폰트(serif/sans-serif)가 존재하지 않아서 라이브 올라가면 깨질거에요

> 재현해보기: [ベクターフィッシュ | 글리프](https://withglyph.com/polaris_observatory/956482871)

시스템 글꼴 sans-serif, serif 를 fallback 으로 설정했습니다.

얼마 전 퇴사 후 갖고 있는 Doppler 및 Tailscale 계정이 없어 PR Preview 를 통해 검수하고자 우선 Draft PR 로 올려둡니다.

---

PS. satori 를 같이 사용 중인 opengraph 이미지 생성 기능에서는 `KoPubWorldDotum` 글꼴 덕에 한자를 제대로 렌더링하고 있음을 확인했습니다.
https://glyph.pub/assets/fonts/ 이 CDN 경로에 KoPubWorld 바탕체 미디움 OTF 인 `KoPubWorldBatangMedium.otf`가 없어 추가 부탁드리겠습니다. 테스트 시에 참고 부탁드립니다.

KoPubWorldBatangMedium font-weight 은 아래 pubspec 를 참고했습니다.
https://github.com/penxle/withglyph/blob/e52dffe3/apps/mobile/pubspec.yaml#L132-L133
@mu-hun mu-hun force-pushed the font-family-fallback branch from 36eaa0e to d6933b7 Compare December 26, 2024 07:07
Copy link
Contributor Author

@mu-hun mu-hun left a comment

Choose a reason for hiding this comment

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

스크린샷 2024-12-26 오후 2 53 36

문제가 그대로 방치되어 있어 @robin-maki 님 코멘트를 바탕으로 제대로 고치게 되었습니다. @devunt 확인 부탁드립니다. 🙇🏻

Comment on lines 190 to 195
fontFamily: `${font}, ${font === 'Pretendard' ? 'sans-serif' : 'serif'}`,
fontFamily: `${font}, ${isSansSerif ? 'KoPubWorldDotum' : 'KoPubWorldBatang'}`,
Copy link
Contributor Author

@mu-hun mu-hun Dec 26, 2024

Choose a reason for hiding this comment

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

satori 환경에 기본 명조체와 고딕체가 없어 KoPubWorld 돋움 및 바탕체를 폴벡 폰트로 수정했습니다.

KoPubWorldBatangMedium font-weight 은 아래 pubspec 를 참고했습니다.
https://github.com/penxle/withglyph/blob/e52dffe3/apps/mobile/pubspec.yaml#L132-L133

테스트 시에 참고 사항

https://glyph.pub/assets/fonts/ CDN 경로에 KoPubWorld 바탕체 미디움 OTF 인 KoPubWorldBatangMedium.otf가 없어 추가 부탁드리겠습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preview 👀👀👀
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants