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

fix: 폰트 preload 에러를 개선한다 #909

Merged
merged 1 commit into from
Oct 28, 2023
Merged

fix: 폰트 preload 에러를 개선한다 #909

merged 1 commit into from
Oct 28, 2023

Conversation

kyw0716
Copy link
Member

@kyw0716 kyw0716 commented Oct 26, 2023

📄 Summary

preload에 다른 설정도 추가해줘야 제대로 동작하는 것을 확인해 해당 설정을 추가해주었습니다.

🕰️ Actual Time of Completion

1분

🙋🏻 More

참고링크

🚀 Storybook

https://storybook.carffe.in/

@kyw0716 kyw0716 added 🚑 버그 예기치 않은 문제 또는 의도하지 않은 동작입니다 FE 프론트엔드 관련 이슈입니다 🚨 긴급 제일 최우선으로 해결해야 할 사항입니다 labels Oct 26, 2023
@kyw0716 kyw0716 self-assigned this Oct 26, 2023
@kyw0716 kyw0716 temporarily deployed to test October 26, 2023 06:09 — with GitHub Actions Inactive
Copy link
Member

@gabrielyoon7 gabrielyoon7 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

Comment on lines 49 to 57
<link
href="https://fonts.googleapis.com/css2?family=Noto+Sans+KR:wght@400;500;700&display=swap"
rel="preload"
as="font"
as="style"
/>
<link
href="https://fonts.googleapis.com/css2?family=Noto+Sans+KR:wght@400;500;700&display=swap"
rel="stylesheet"
/>
Copy link
Collaborator

@feb-dain feb-dain Oct 26, 2023

Choose a reason for hiding this comment

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

Suggested change
<link
href="https://fonts.googleapis.com/css2?family=Noto+Sans+KR:wght@400;500;700&display=swap"
rel="preload"
as="font"
as="style"
/>
<link
href="https://fonts.googleapis.com/css2?family=Noto+Sans+KR:wght@400;500;700&display=swap"
rel="stylesheet"
/>
<link
href="https://fonts.googleapis.com/css2?family=Noto+Sans+KR:wght@400;500;700&display=swap"
rel="preload"
as="font"
onload=“this.onload=null; this.rel=‘stylesheet’”
/>

혹시 링크 두 개 대신 이렇게 해도 되는지 확인해줄 수 있나요??

@feb-dain feb-dain merged commit f765cf0 into develop Oct 28, 2023
10 checks passed
@feb-dain feb-dain deleted the fix/908 branch October 28, 2023 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 관련 이슈입니다 🚑 버그 예기치 않은 문제 또는 의도하지 않은 동작입니다 🚨 긴급 제일 최우선으로 해결해야 할 사항입니다
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants