Skip to content

[문서] 문서 1.1 ~ 1.7 병합충돌 해결#1850

Open
BHyeonKim wants to merge 9 commits into
javascript-tutorial:2026-en-mergefrom
BHyeonKim:sync-c23
Open

[문서] 문서 1.1 ~ 1.7 병합충돌 해결#1850
BHyeonKim wants to merge 9 commits into
javascript-tutorial:2026-en-mergefrom
BHyeonKim:sync-c23

Conversation

@BHyeonKim
Copy link
Copy Markdown

@BHyeonKim BHyeonKim commented May 13, 2026

요약

문서[document] 파트의 병합 충돌을 해결했습니다

  • 1.1 브라우저 환경과 다양한 명세서
  • 1.2 DOM 트리
  • 1.3 DOM 탐색하기
  • 1.4 getElement*, querySelector*로 요소 검색하기
  • 1.5 주요 노드 프로퍼티
  • 1.6 속성과 프로퍼티
  • 1.7 문서 수정하기

1.7 문서 수정하기에 미 번역된 부분이 있어서 다음과 같이 번역했습니다.

Also it is important to set a new interval in `clockStart()` only when the clock is not running. Otherways clicking the start button several times would set multiple concurrent intervals. Even worse - we would only keep the `timerID` of the last interval, losing references to all others. Then we wouldn't be able to stop the clock ever again! Note that we need to clear the `timerID` when the clock is stopped in the line `(**)`, so that it can be started again by running `clockStart()`.

또한 `clockStart()`에서 새 인터벌을 설정할 때는 시계가 실행 중이지 않을 때만 해야 합니다. 그렇지 않으면 시작 버튼을 여러 번 클릭할 경우 여러 인터벌이 동시에 설정됩니다. 더 심각한 문제는 마지막 인터벌의 `timerID`만 보관하게 되어 나머지 인터벌에 대한 참조를 잃게 된다는 점입니다. 그러면 시계를 다시는 멈출 수 없게 됩니다! `(**)`로 표시한 줄에서 시계가 멈출 때 `timerID`를 초기화해야 한다는 점도 유의하세요. 이렇게 해야 나중에 `clockStart()`를 실행해 시계를 다시 시작할 수 있습니다.

연관 이슈

(fix #일이삼)

Pull Request 체크리스트

TODO

  • 번역 규칙을 확인하셨나요?
    • 줄 바꿈과 단락을 '원문과 동일하게' 유지하셨나요?
    • 맞춤법 검사기로 맞춤법을 확인하셨나요?
    • 마크다운 문법에 사용되는 공백(스페이스), 큰따옴표("), 작은따옴표('), 대시(-), 백틱(`) 등의 특수문자는 그대로 두셨나요?
  • 로컬 서버 세팅 후 최종 결과물을 확인해 보셨나요?
  • PR 하나엔 번역문 하나만 넣으셨나요?
  • 의미 있는 커밋 메시지를 작성하셨나요?
    • 예시
      • [프락시] 번역
      • [프락시] 과제 번역
      • [if문과 조건부 연산자 '?'] 리뷰
      • [주석] 2차 리뷰
      • [Date 객체와 날짜] 번역

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 13, 2026

CLA assistant check
All committers have signed the CLA.

@BHyeonKim
Copy link
Copy Markdown
Author

@snowari 리뷰 부탁드려요~

Copy link
Copy Markdown

@snowari snowari left a comment

Choose a reason for hiding this comment

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

upstream 원본 내용이 빠진 부분이 있어서 확인 한번 해주세요!

Comment thread 2-ui/1-document/05-basic-dom-node-properties/article.md
Comment thread 2-ui/1-document/05-basic-dom-node-properties/article.md
@BHyeonKim BHyeonKim requested a review from snowari May 17, 2026 09:07
Copy link
Copy Markdown

@snowari snowari left a comment

Choose a reason for hiding this comment

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

수정된 부분 확인했습니다!

@Violet-Bora-Lee
Copy link
Copy Markdown
Member

PR의 단위가 너무 큽니다. 가능하시다면 파일 단위로 PR을 쪼개주시면 감사하겠습니다. 아니면 현재 7개의 수정을 한 커밋에 넣으셨는데, 그 커밋을 쪼개서 올려주시면 리뷰에 용이할것같습니다.

interactive rebase <-- 커밋 쪼개기

@BHyeonKim
Copy link
Copy Markdown
Author

@Violet-Bora-Lee
interactive rebase를 사용하여 기존 커밋을 7개의 커밋으로 쪼개서 푸시했습니다.
감사합니다.

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.

5 participants