Skip to content

[CSS 애니메이션] 해답 번역#570

Merged
Violet-Bora-Lee merged 1 commit intojavascript-tutorial:masterfrom
adk1245:translate2
Jul 5, 2020
Merged

[CSS 애니메이션] 해답 번역#570
Violet-Bora-Lee merged 1 commit intojavascript-tutorial:masterfrom
adk1245:translate2

Conversation

@adk1245
Copy link
Copy Markdown
Contributor

@adk1245 adk1245 commented Jun 11, 2020

Pull Request 체크리스트

TODO

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

Copy link
Copy Markdown
Member

@Violet-Bora-Lee Violet-Bora-Lee left a comment

Choose a reason for hiding this comment

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

모범사례 확인하시어 수정 부탁드립니다.
아직 컨벤션이 익숙하지 않으신것 같은데요,
익숙해지실때까지 한번에 하나만 작업부탁드릴게요.

@@ -1,7 +1,7 @@
We need to choose the right Bezier curve for that animation. It should have `y>1` somewhere for the plane to "jump out".
�츮�� �� �ִϸ��̼ǿ� �˸��� ������ ��� �����ؾ� �մϴ�. ����Ⱑ "jump out"�Ϸ��� ��򰡿� `y>1`�� �־�� �մϴ�.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@javascript-translate-bot
Copy link
Copy Markdown

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

@adk1245
Copy link
Copy Markdown
Contributor Author

adk1245 commented Jun 17, 2020

컨벤션 숙지 하지 못하고 작업한 점 죄송합니다. 수정 했으니 확인 부탁드립니다!!

@Violet-Bora-Lee
Copy link
Copy Markdown
Member

@ohdajung @adk1245 이 두 계정이 동일한 분이신가요?

@Violet-Bora-Lee Violet-Bora-Lee merged commit d982151 into javascript-tutorial:master Jul 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants