1-js/08-prototypes/02-function-prototype/article.md 리뷰#228
Closed
SimEunJu wants to merge 5 commits intojavascript-tutorial:masterfrom
SimEunJu:master
Closed
1-js/08-prototypes/02-function-prototype/article.md 리뷰#228SimEunJu wants to merge 5 commits intojavascript-tutorial:masterfrom SimEunJu:master
SimEunJu wants to merge 5 commits intojavascript-tutorial:masterfrom
SimEunJu:master
Conversation
|
|
Member
|
이 글은 다른 기여자분이 작성해 주신 거라 합의되지 않은 내용이 있을 수 있어요. 이 글 리뷰하는데는 저도 시간이 꽤 걸릴 것같아서 천천히 보도록 하겠습니다. |
Violet-Bora-Lee
requested changes
Oct 1, 2019
Member
Violet-Bora-Lee
left a comment
Violet-Bora-Lee
left a comment
There was a problem hiding this comment.
리뷰 감사합니다.
대시보드에 제가 아래와 같은 코멘트 남겨두었는데, 확인해주시고 응답 부탁드릴게요.
해당 글은, 번역 컨벤션이 다듬어지기 훨씬 이전에 작성된 글이어서 수정해야할 것들이 많습니다. 이 글 보다는 바로 윗 글(프로토타입 상속)에 관한 글을 리뷰해주시길 부탁드릴게요.
| 만약 생성 후에 `F.prototype` 프로퍼티가 변한다면(`F.prototype=<다른 객체>`), `new F`에 의해 생성된 새로운 객체는 `[[Prototype]]`으로 다른 객체를 가질 것입니다. 그러나 이미 생성된 객체는 이전 객체를 유지합니다. | ||
| ``` | ||
|
|
||
|
|
Member
There was a problem hiding this comment.
불필요한 줄이 들어가있어서, line이 맞지 않습니다.
리드미를 읽어보시고, 수정해주세요.
https://github.com/javascript-tutorial/ko.javascript.info#%EB%B2%88%EC%97%AD-%EA%B7%9C%EC%B9%99
|
Please make the requested changes. After it, add a comment "/done". |
Member
|
번역 컨벤션이 합의되기 전, 한참전에 작성된 글이어서 리뷰가 불가능 상태였습니다. 제가 다시 번역하여 commit하였습니다. 본 PR은 클로징 하겠습니다. 감사합니다. |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
#226
1차 리뷰를 진행하였습니다.
용어집에서는 default를 default문으로 번역하고 있는데,
본문의 default property와는 어감이 다른 것 같습니다ㅠㅠ
일단 기본값으로 번역하기는 하였으나, 변경사항이 있으면 알려주세요!