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

Translate: 03-pages > 01-building-your-application > 05-optimizing > 02-fonts.mdx #284

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

shyuuuuni
Copy link

@shyuuuuni shyuuuuni commented Jun 27, 2023

기여자용

문서 개선

comment

  • docs/03-pages/05-optimizing/02-fonts.mdxdocs/02-app/...의 동일 파일을 참조하기 때문에 해당 파일에서 작업했습니다.
  • 폰트(font)를 글꼴로 번역했는데, 외래어 그대로 폰트로 사용하는건 어떻게 생각하시나요?
  • React, Next.js에서 pre + rendering, loading과 같은 단어가 자주 나타나는데, 통일되게 용어 정리를 하는건 어떨까요?

Progress

@haileyport
Copy link
Contributor

ref #112

@g2hhh2ee
Copy link
Contributor

g2hhh2ee commented Jul 3, 2023

안녕하세요:)
docs/03-pages/05-optimizing/02-fonts.mdx 을 할당받으셔서 해당 파일을 제외하고는 지워주셔야 합니다 🥲
언급해주신대로 docs/02-app/... 부분은 다른 분께서 할당받으신 부분입니다

추가로 하시고 싶은 티켓이 있다면 새로운 티켓 신청 부탁드립니다

@shyuuuuni
Copy link
Author

shyuuuuni commented Jul 3, 2023

안녕하세요:) docs/03-pages/05-optimizing/02-fonts.mdx 을 할당받으셔서 해당 파일을 제외하고는 지워주셔야 합니다 🥲 언급해주신대로 docs/02-app/... 부분은 다른 분께서 할당받으신 부분입니다

추가로 하시고 싶은 티켓이 있다면 새로운 티켓 신청 부탁드립니다

docs/03-pages/05-optimizing/02-fonts.mdx 부분을 보면

---
title: Font Optimization
nav_title: Fonts
description: Optimize your application's web fonts with the built-in `next/font` loaders.
source: 'app/building-your-application/optimizing/fonts'
---

{/* DO NOT EDIT. The content of this doc is generated from the source above. To edit the content of this page, navigate to the source page in your editor. You can use the `<PagesOnly>Content</PageOnly>` component to add content that is specific to the Pages Router. Any shared content should not be wrapped in a component. */}

위와 같이 'app/building-your-application/optimizing/fonts'에 있는 코드에서 해당 콘텐츠를 수정하라고 나와있는데, <PagesOnly> 부분도 수정하면 안되는 것인지 궁금합니다.

@g2hhh2ee
Copy link
Contributor

g2hhh2ee commented Jul 3, 2023

---
title: Font Optimization
nav_title: Fonts
description: Optimize your application's web fonts with the built-in `next/font` loaders.
source: 'app/building-your-application/optimizing/fonts'
---

pages 파일 내의 위 메타데이터 부분만 수정해주시면 됩니다 :)
app 파일 내의 <PagesOnly> 부분도 각 app 작업자에게 할당됩니다

@g2hhh2ee g2hhh2ee added the duplicate App, Pages 공통 문서 label Jul 14, 2023
@ChanghyeonYoon
Copy link
Contributor

리뷰 이후 작업이 몇일 동안 이뤄지지 않아 9월 13일까지 작업이 없을 경우 이슈를 close하고 새로운 분에게 할당하겠습니다! @shyuuuuni

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation 문서 작업 duplicate App, Pages 공통 문서 In Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

02-fonts.mdx
4 participants