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

"Adopting Concurrent Mode" 번역 #182

Merged
merged 9 commits into from
Dec 3, 2019
Merged

"Adopting Concurrent Mode" 번역 #182

merged 9 commits into from
Dec 3, 2019

Conversation

junsikshim
Copy link
Contributor

@netlify
Copy link

netlify bot commented Nov 7, 2019

Deploy preview for ko-reactjs-org ready!

Built with commit 62ee753

https://deploy-preview-182--ko-reactjs-org.netlify.com

@netlify
Copy link

netlify bot commented Nov 7, 2019

Deploy preview for ko-reactjs-org ready!

Built with commit 6e83dd0

https://deploy-preview-182--ko-reactjs-org.netlify.com

>
>This documentation is aimed at early adopters and people who are curious. If you're new to React, don't worry about these features -- you don't need to learn them right now.
>이 문서는 얼리 어답터와 궁금해하시는 분을 대상으로 합니다. React를 처음 쓰신다면 이 기능들에 대해 신경 쓰지 마세요. 당장 익히실 필요는 없습니다.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
>이 문서는 얼리 어답터와 궁금해하시는 분을 대상으로 합니다. React를 처음 쓰신다면 이 기능들에 대해 신경 쓰지 마세요. 당장 익히실 필요는 없습니다.
>이 문서는 얼리 어답터와 궁금해하시는 분을 대상으로 합니다. React를 처음 쓰신다면 이 기능들에 대해 신경 쓰지 마세요. 당장 익힐 필요는 없습니다.

* **Concurrent Mode:** `ReactDOM.createRoot(rootNode).render(<App />)`. It is currently experimental. In the future, after it stabilizes, we intend to make it the default React mode. This mode enables *all* the new features.
* **레거시 모드** `ReactDOM.render(<App />, rootNode)`. 현재 React 앱들이 사용하고 있는 모드입니다. 아직 가까운 미래에 레거시 모드를 없앨 계획은 없지만, 신규 기능들을 사용할 수는 없을 겁니다.
* **Blocking 모드** `ReactDOM.createBlockingRoot(rootNode).render(<App />)`. 현재 실험 중인 상태입니다. Concurrent 모드의 일부 기능을 제공할 수 있는 첫 번째 마이그레이션 단계입니다.
* **Concurrent 모드** `ReactDOM.createRoot(rootNode).render(<App />)`. 현재 실험 중인 상태입니다. 안정된 후엔 디폴트 React 모드로 만들 예정입니다. 이 모드에선 *모든* 신규 기능을 사용할 수 있습니다.
Copy link
Member

Choose a reason for hiding this comment

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

디폴트도 적절한 한국어로 대응할 수 있을 것 같습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

'디폴트 React 모드' -> 'React 기본 모드'로 수정하겠습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

여기의 '레거시'도 아래의 논의 이후에 일치시키도록 하겠습니다.

@@ -105,26 +105,27 @@ You can think of the Blocking Mode as a "gracefully degraded" version of the Con

<div id="feature-table">

| |Legacy Mode |Blocking Mode |Concurrent Mode |
| |레거시 모드 |Blocking 모드 |Concurrent 모드 |
Copy link
Member

Choose a reason for hiding this comment

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

레거시도 영문 그대로 사용하는게 어떨까요? #2 에서 논의된 적이 없는 것 같아서 의견 달아봅니다.

Copy link
Member

Choose a reason for hiding this comment

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

@hg-pyun 영문 그대로 사용하는데에 동의합니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

https://ko.reactjs.org/docs/context.html 에선 '예전'을 사용하고 있고,
https://ko.reactjs.org/docs/refs-and-the-dom.html,
https://ko.reactjs.org/docs/strict-mode.html,
https://ko.reactjs.org/docs/react-component.html 에선 '레거시'를 사용하고 있습니다.

'Legacy Mode'라는 대명사로 볼 수 있다면 영문으로 사용하는 것도 좋겠지만,
그냥 예전 모드를 지칭하는 거면 다른 페이지들과 맞추는 게 나을 것 같기도 합니다.

의견 부탁드립니다.

Copy link
Member

Choose a reason for hiding this comment

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

Legacy Mode라는 고유 이름을 그대로 사용하는게 좋을것 같습니다. cc: @hg-pyun @taehwanno

Copy link
Member

@hg-pyun hg-pyun Nov 14, 2019

Choose a reason for hiding this comment

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

저도 고유명사 그대로 사용하는 것에 동의합니다!

Copy link
Member

Choose a reason for hiding this comment

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

이번에 Concurrent Mode가 릴리즈 되면서 새롭게 구분하는 개념이라 그대로 사용하는 것에 동의합니다 :)

@junsikshim
Copy link
Contributor Author

최종 확인 부탁드립니다~

@rkdmf1026 rkdmf1026 mentioned this pull request Nov 20, 2019
6 tasks
@taehwanno
Copy link
Member

72775e9 커밋으로 인해 충돌이 발생하네요 ㅎㅎ 충돌 해결 부탁드려요!

Copy link
Member

@taehwanno taehwanno left a comment

Choose a reason for hiding this comment

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

9d878b4 커밋과 충돌 해결했고 작은 수정사항 직접 커밋해서 푸쉬했습니다. 고생하셨습니다!

Copy link
Member

@hg-pyun hg-pyun left a comment

Choose a reason for hiding this comment

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

LGTM

@taehwanno taehwanno merged commit 3059513 into reactjs:master Dec 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants