Refactor/file route #18
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bepyan
requested review from
xilucks,
WooWan,
delphox60 and
9keyyyy
as code owners
|
The latest updates on your projects.
|
invibee
reviewed
Jul 31, 2024
Comment on lines
+13
to
+18
| <link | ||
| rel="stylesheet" | ||
| as="style" | ||
| crossOrigin="anonymous" | ||
| href="https://cdn.jsdelivr.net/gh/orioncactus/[email protected]/dist/web/variable/pretendardvariable-dynamic-subset.min.css" | ||
| /> |
There was a problem hiding this comment.
(창완): assets/fonts 에도 pretendard 폰트가 있는데 요것도 필요한가요?
There was a problem hiding this comment.
위 설명에 있긴한데,
서버 부하를 줄이는 차원에서 외부 cdn 쓰는 코드 유지했어요.
부하로 문제 될 건 없긴한데.. 폰트가 서버 대역폭 다 차지하는거 너무 맴찟이에요
https://vercel.com/invis-projects/~/usage#Fast%20Data%20Transfer
There was a problem hiding this comment.
서버 대역폭 문제로 cdn으로 옮기는 건 이해했는데, assets/fonts 에도 pretendard 폰트가 있는데 중복 아닌가 해서요!
There was a problem hiding this comment.
일단 요정도까지 정리하고 추후 개선해보면 어떨지,
- local font는 서버 대역폭 부하가 있다.
- cdn font는 서버 대역폭 부하가 없지만 local font보다 느리다.
- 실서비스는 폰트가 빠르게 로딩되는 것이 좋기에 local font를 쓴다.
- 내부서비스는 빠르게 로딩되는 것이 중요하지 않기에 cdn font를 쓴다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
수정된 내용
전체적으로 파일 베이스를 조금 다듬었습니다.
작업 중인 브랜치와 무조건 충돌될거 같은데 조금 양해 구해요.. 🙈
1.
/plaground->/pg쉽게 접근하기 위해 네이밍을 줄였어요.
2. html lang
kr->ko한국어의 언어 태그는 ko입니다.
3.
layout.tsx분기서로 영향을 주지 않도록
layout.tsx을 관심사 별로 수정했습니다.4.
/pg프리텐다드 적용이전에 적용이 안되고 있었는데..
서버 부하 줄이는 차원에서 외부 cdn으로 폰트 파일 적용했습니다.
5. 폰트 파일
/public>/assets로 이동assets에 두면 최적화되는지 보려했는데 폰트 파일을 따로 프로세싱을 하지 않나보네요.
추후 public에 있는 ut 이미지들 assets로 옮겨 주세용
L 공용으로 쓰지 않는 리소스는 app route에서
/_assets에 모아두면 어떨지 싶소.