Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[내용번역] Part 2. 3.5 Scrolling#739

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
marco0332 wants to merge2 commits intojavascript-tutorial:master
base:master
Choose a base branch
Loading
frommarco0332:scrolling

Conversation

marco0332
Copy link
Contributor

Pull Request 체크리스트

TODO

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

@CLAassistant
Copy link

CLAassistant commentedAug 25, 2020
edited
Loading

CLA assistant check
All committers have signed the CLA.

Comment on lines +6 to +7
-문서에서 사용자가 위치한 곳에 따라 추가 제어 요소 또는 정보를 표시하거나 숨긴다.
-사용자가 페이지 끝까지 스크롤을 내릴 경우 더 많은 데이터를 불러온다.

Choose a reason for hiding this comment

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

경어체를 유지해주세요.

명사형으로 끝나는 문장은 경어체가 아니어도 됩니다.

~숨김,~불러옴

```

The`scroll`event works both on the`window` and on scrollable elements.
`scroll`이벤트는`window`와 스크롤 가능한 요소에서 동작합니다.

Choose a reason for hiding this comment

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

스크롤 가능한 요소라고 번역하면 조금 헷갈릴수있으니 한영병기해주는게 어떨지 제의드려봅니다.

스크롤 가능한 요소(scrollable element)


We can't prevent scrolling by using`event.preventDefault()` in `onscroll` listener, because it triggers *after* the scroll has already happened.
`onscroll` 리스너에서`event.preventDefault()`로 스크롤을 막는 것은 onscroll 리스너가 스크롤이 이미 발생한 뒤에 동작하기 때문에 불가능합니다.

Choose a reason for hiding this comment

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

  1. 원문에서 after를 강조했는데, 그부분이 빠져있네요.
  2. onscroll리스너가 스크롤이 이미 발생한 뒤에 <-- 이부분이 이상합니다. 문장 확인해주세요.

@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@Violet-Bora-Lee
Copy link
Member

코멘트 확인해주시고, 전체적으로 퇴고 여러번 부탁드립니다.

marco0332 reacted with thumbs up emojimarco0332 reacted with heart emoji

@Violet-Bora-Lee
Copy link
Member

안녕하세요. 충돌도 있고 오래된 PR이라서 본 PR은 클로즈하도록 하겠습니다.

재 번역 의사 있으시면 4월 30일까지 본 PR에 댓글 부탁드릴게요.
아니라면 작업자 전환을 하도록 하겠습니다.

기여해주셔서 감사합니다 :)

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@Violet-Bora-LeeViolet-Bora-LeeViolet-Bora-Lee requested changes

Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@marco0332@CLAassistant@javascript-translate-bot@Violet-Bora-Lee

[8]ページ先頭

©2009-2025 Movatter.jp