-
Notifications
You must be signed in to change notification settings - Fork 41
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(app): 앱에 필요한 폰트들을 일부 추가하고, 전역 스타일링 설정을 추가합니다. #79
Conversation
apps/react-world/src/main.tsx
Outdated
import { Global } from '@emotion/react'; | ||
import globalStyles from '../styles/globalStyles'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이것도 팁은, routes와 같이 컴포넌트로 분리해서 globalStyles를 main.tsx에서 숨겨버리는 것입니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오 그렇군요.. 아래처럼 Routes 안에 같이 넣는 걸 의미하신 거죠~? 이렇게 수정해두었습니다!
export const Routes = () => {
return (
<Layout>
<Global styles={globalStyles} />
<RouterProvider router={router} />;
</Layout>
);
};
const globalStyles = css` | ||
@font-face { | ||
font-family: 'Titillium Web'; | ||
src: url('/fonts/TitilliumWeb/TitilliumWeb-Regular.ttf') format('truetype'); | ||
font-weight: normal; | ||
font-style: normal; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이렇게 작성되는 폰트는 캐싱이나... 레이지 로드가 되지 않아서 좀 지양되는 편이긴 합니다.
물론 동작에는 아무런 이슈가 없습니다!
간혹 폰트 용량이 엄청 크기 때문에... 사용자가 fouc 현상을 마주하게 될 수 있어요.
index.html이나 preload 하는 속성을 활용하면 더 좋을 것 같아요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이렇게 작성되는 폰트는 캐싱이나... 레이지 로드가 되지 않아서 좀 지양되는 편이긴 합니다. 물론 동작에는 아무런 이슈가 없습니다!
간혹 폰트 용량이 엄청 크기 때문에... 사용자가 fouc 현상을 마주하게 될 수 있어요. index.html이나 preload 하는 속성을 활용하면 더 좋을 것 같아요!
오 안그래도 뭔가 다른 방법은 없으려나 싶기는 했는데.. 자료 공유 감사합니다! 요건 이번 PR 말고 추후 작업으로 개선해봐도 좋을 것 같습니다 ㅎㅎ
일단 내용은 읽어봤는데 의존성 추가하는데 까지는 바로 할 것 같은데 현재 상태에서 추가 세팅을 어떤 부분을 더 해야 하는지 좀 더 찾아봐야할 것 같네요..!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
향후 아래 방향으로 다시 트라이 해보겠습니다!
index.html이나 preload 하는 속성을 활용하면 더 좋을 것 같아요!
0f2acf8
to
6672475
Compare
c3ba8ab
to
287462e
Compare
9abbdfe
to
6a956f7
Compare
📌 이슈 링크
📖 작업 배경
🛠️ 구현 내용
TitilliumWeb
폰트와,SourceSansPro
폰트를 public/fonts 폴더에 각각 추가합니다. 일단 당장 필요해 보이는 것들만 넣었습니다.globalStyles.tsx
파일에 emotion 기반으로 폰트 관련 전역 스타일링 코드를 추가합니다.main.tsx
에서 전역 스타일링을 적용합니다.main.css
쪽에서 전역 스타일링으로 넣어도 될 만한 부분들을 뽑아globalStyles.tsx
쪽에 적용해 주었습니다.💡 참고사항
🖼️ 스크린샷
데모 페이지
작업 중인 페이지