-
Notifications
You must be signed in to change notification settings - Fork 143
translate: Dom Manipulation(~ end) #183
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
Conversation
* first translation(9/7) * 2차 검토, 맞춤법검사, lint 검사(9/8) * 오타 재검토
몇 가지 제안 남겨두었습니다 |
* 제안 반영 Co-authored-by: GuyeolJeong <[email protected]>
* 제안 반영 Co-authored-by: GuyeolJeong <[email protected]>
제안해주신 내용 확인했습니다! 번역 누락까지 지적 감사합니다! |
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.
고생하셨습니다 :)
* 코드 리뷰 제안 반영 Co-authored-by: Kibeom Kwon <[email protected]>
* 제안반영 Co-authored-by: Kibeom Kwon <[email protected]>
* 제안반영 Co-authored-by: Kibeom Kwon <[email protected]>
말씀해주신 제안사항이 매끄러워 모두 반영했습니다!🙇🏻♀️ |
* line 197 번역 누락 수정
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.
👍 👍
PR전에 아래의 사항들을 확인하셨나요? 확인하셨다면, 체크해주세요!
PR에 대한 간단한 내용 설명을 부탁드립니다!
(ref: Dom Manipulation 번역 #117) Dom Manipulation @sooyun429 님이 맡으신 부분 이후부터 끝까지 번역 진행하였습니다.
번역 리뷰시 중점적으로 봐주셨으면 하는 부분 함께 첨부하겠습니다.
Node
,Element
인터페이스에 대한 이야기가 중점적으로 많이 등장하는데, 이 부분은 인터페이스명 그대로 표기하였습니다. 중간중간에 인터페이스를 지칭하는 것이 아닌 것 같은 부분은 노드, 요소로 번역했으니 이 부분에 대해 적절치 못한 부분이 있으면 의견 부탁드립니다!line137
의 After capturing thediv
element 부분과line178
에 의역한 부분이 조금 있어서 중점적으로 봐주시면 좋을 것 같습니다!