Skip to content
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

feat: translate window/scrollby in ko #10976

Merged
merged 1 commit into from
Jan 18, 2023

Conversation

sukvvon
Copy link
Contributor

@sukvvon sukvvon commented Jan 7, 2023

Description

translate web/api/window/scrollby in ko

Motivation

Window.scrollBy() page is not translated in ko so i translated

Additional details

Related issues and pull requests

@sukvvon sukvvon requested a review from a team as a code owner January 7, 2023 06:39
@sukvvon sukvvon requested review from hochan222 and removed request for a team January 7, 2023 06:39
@github-actions github-actions bot added the l10n-ko Issues related to Korean content. label Jan 7, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jan 7, 2023

Preview URLs

Flaws (3)

URL: /ko/docs/Web/API/Window/scrollBy
Title: Window.scrollBy()
Flaw count: 3

  • macros:
    • /ko/docs/Web/API/Window/scroll does not exist but fell back to /en-US/docs/Web/API/Window/scroll
    • /ko/docs/Web/API/Window/scrollByLines does not exist but fell back to /en-US/docs/Web/API/Window/scrollByLines
    • /ko/docs/Web/API/Window/scrollByPages does not exist but fell back to /en-US/docs/Web/API/Window/scrollByPages

Copy link
Member

@hochan222 hochan222 left a comment

Choose a reason for hiding this comment

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

@sukvvon 님 안녕하세요.

기여해주셔서 감사합니다. 좋은 하루 보내세요 :)

\- 혹은 -

- `options`
- : 다음 매개변수가 포함된 사전.
Copy link
Member

@hochan222 hochan222 Jan 18, 2023

Choose a reason for hiding this comment

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

Suggested change
- : 다음 매개변수가 포함된 사전.
- : 다음 매개변수가 포함된 사전.

A dictionary containing the following parameters:

영어 원문일때는 괜찮아 보였는데 번역이 사전이 되어버리니까.. 뭔가 의미전달이 잘 안되는것같기도 하네요.. 키값을 갖는 객체.. 음.. 오랜시간 고민했지만 결국 사전으로 두는게 좋아보입니다.


명세에 정의가 dictionary로 되어있기도 하고, javascript에서 익숙하지 않을뿐이지 사전이라고 많이 사용하네요..!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

저도 번역하면서 같은 생각을 했답니다 ㅎㅎ 😄
좋은 하루 보내세요 :)

@hochan222 hochan222 merged commit a8fd27b into mdn:main Jan 18, 2023
@sukvvon sukvvon deleted the ko_translate_window-scrollby branch January 18, 2023 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-ko Issues related to Korean content.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants