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

번역 완료 #391

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

번역 완료 #391

wants to merge 1 commit into from

Conversation

nada1221
Copy link

@nada1221 nada1221 commented Jul 2, 2023

기여자용

문서 개선

Progress

close #36

@haileyport
Copy link
Contributor

ref #36

Copy link
Contributor

@chaejunlee chaejunlee left a comment

Choose a reason for hiding this comment

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

이미 머지된 다른 문서들을 참고해서 포맷을 맞춰주시면 좋을 것 같습니다 😀!!

리뷰 남겨드렸으니 확인 후 반영 부탁 드립니다!


## Installing Tailwind
<!-- is a utility-first CSS framework that works exceptionally well with Next.js. -->
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<!-- is a utility-first CSS framework that works exceptionally well with Next.js. -->

Install the Tailwind CSS packages and run the `init` command to generate both the `tailwind.config.js` and `postcss.config.js` files:
## Tailwind 설치

<!-- Install the Tailwind CSS packages and run the `init` command to generate both the `tailwind.config.js` and `postcss.config.js` files: -->
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<!-- Install the Tailwind CSS packages and run the `init` command to generate both the `tailwind.config.js` and `postcss.config.js` files: -->


<!-- Install the Tailwind CSS packages and run the `init` command to generate both the `tailwind.config.js` and `postcss.config.js` files: -->

Tailwind CSS 패키지들을 설치 후 `tailwind.config.js`파일과 `postcss.config.js`파일을 생성하기 위해 `init` 커맨드를 사용합니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Tailwind CSS 패키지들을 설치 후 `tailwind.config.js`파일과 `postcss.config.js`파일을 생성하기 위해 `init` 커맨드를 사용합니다.
Tailwind CSS 패키지들을 설치 후 `tailwind.config.js` 파일과 `postcss.config.js` 파일을 생성하기 위해 `init` 커맨드를 사용합니다.

Comment on lines +34 to +35
<!--
Inside `tailwind.config.js`, add paths to the files that will use Tailwind CSS class names: -->
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<!--
Inside `tailwind.config.js`, add paths to the files that will use Tailwind CSS class names: -->

'./pages/**/*.{js,ts,jsx,tsx,mdx}',
'./components/**/*.{js,ts,jsx,tsx,mdx}',

// Or if using `src` directory:
//또는 `src` 디렉토리를 사용하는 경우에는:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
//또는 `src` 디렉토리를 사용하는 경우에는:
// 또는 `src` 디렉토리를 사용하는 경우라면

@@ -86,7 +101,7 @@ export default function RootLayout({
```

```jsx filename="app/layout.js" switcher
// These styles apply to every route in the application
// 이 스타일은 애플리케이션의 모든 루트에 적용됩니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// 이 스타일은 애플리케이션의 모든 루트에 적용됩니다.
// 이 스타일은 애플리케이션의 모든 라우트에 적용됩니다.


After installing Tailwind CSS and adding the global styles, you can use Tailwind's utility classes in your application.
<!-- After installing Tailwind CSS and adding the global styles, you can use Tailwind's utility classes in your application. -->
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<!-- After installing Tailwind CSS and adding the global styles, you can use Tailwind's utility classes in your application. -->

@@ -119,6 +136,8 @@ export default function Page() {
}
```

## Usage with Turbopack
## 터보 팩과 함께 사용
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
## 터보 팩과 함께 사용
## 터보팩과 함께 사용

## Usage with Turbopack
## 터보 팩과 함께 사용

<!-- As of Next.js 13.1, Tailwind CSS and PostCSS are supported with [Turbopack](https://turbo.build/pack/docs/features/css#tailwind-css). -->
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<!-- As of Next.js 13.1, Tailwind CSS and PostCSS are supported with [Turbopack](https://turbo.build/pack/docs/features/css#tailwind-css). -->


As of Next.js 13.1, Tailwind CSS and PostCSS are supported with [Turbopack](https://turbo.build/pack/docs/features/css#tailwind-css).
Next.js 13.1부터 Tailwind CSS 및 PostCSS는 [터보 팩](https://turbo.build/pack/docs/features/css#tailwind-css)에서 지원됩니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Next.js 13.1부터 Tailwind CSS 및 PostCSS는 [터보 팩](https://turbo.build/pack/docs/features/css#tailwind-css)에서 지원됩니다.
Next.js 13.1부터 Tailwind CSS 및 PostCSS는 [터보팩](https://turbo.build/pack/docs/features/css#tailwind-css)에서 지원됩니다.

@ChanghyeonYoon
Copy link
Contributor

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

@nada1221
Copy link
Author

nada1221 commented Sep 7, 2023

아 피드백이 왔었군요 수정 진행하고 다시 달겠습니다.

Copy link
Contributor

@yoo-jimin127 yoo-jimin127 left a comment

Choose a reason for hiding this comment

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

@nada1221 님 안녕하세요!
리뷰 반영 최종 확인 및 머지 담당자입니다.
현재 chaejunlee 님께서 작성해주신 리뷰 내용 중 반영되어 있지 않은 부분들이 몇가지 보이는데요,
예를 들어

//또는 `src` 디렉토리를 사용하는 경우에는:

과,

// 이 스타일은 애플리케이션의 모든 루트에 적용됩니다.

를 비롯해 몇가지 리뷰의 경우 반영이 필수적으로 필요해 보입니다!
올려주신 커밋에서 확인이 되지 않아, 한번 더 확인 부탁드립니다 :)
반영 작업 완료해주시면 언급 한 번만 부탁드릴게요!
감사합니다 :) ☺️

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

Successfully merging this pull request may close these issues.

02-tailwind-css.mdx
5 participants