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

[ko] Intl.DisplayNames #14052

Merged
merged 6 commits into from
Aug 1, 2023
Merged

[ko] Intl.DisplayNames #14052

merged 6 commits into from
Aug 1, 2023

Conversation

SnowMarble
Copy link
Contributor

No description provided.

@SnowMarble SnowMarble requested a review from a team as a code owner July 6, 2023 01:50
@SnowMarble SnowMarble requested review from wisedog and removed request for a team July 6, 2023 01:50
@github-actions github-actions bot added the l10n-ko Issues related to Korean content. label Jul 6, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jul 6, 2023

Preview URLs

Flaws (6)

URL: /ko/docs/Web/JavaScript/Reference/Global_Objects/Intl/DisplayNames
Title: Intl.DisplayNames
Flaw count: 6

  • macros:
    • /ko/docs/Web/JavaScript/Reference/Global_Objects/Intl/DisplayNames/DisplayNames does not exist but fell back to /en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/DisplayNames/DisplayNames
    • /ko/docs/Web/JavaScript/Reference/Global_Objects/Intl/DisplayNames/supportedLocalesOf does not exist but fell back to /en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/DisplayNames/supportedLocalesOf
    • /ko/docs/Web/JavaScript/Reference/Global_Objects/Intl/DisplayNames/DisplayNames does not exist but fell back to /en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/DisplayNames/DisplayNames
    • /ko/docs/Web/JavaScript/Reference/Global_Objects/Intl/DisplayNames/of does not exist but fell back to /en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/DisplayNames/of
    • /ko/docs/Web/JavaScript/Reference/Global_Objects/Intl/DisplayNames/resolvedOptions does not exist but fell back to /en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/DisplayNames/resolvedOptions
  • broken_links:
    • Can use the English (en-US) link as a fallback

(comment last updated: 2023-07-09 05:55:40)

Copy link
Contributor

@wisedog wisedog left a comment

Choose a reason for hiding this comment

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

오랜만에 기여 감사합니다. 몇 가지 의견을 남겼으니 확인 부탁드릴께요.

languageNames.of("en-US"); // "American English"
languageNames.of("zh-TW"); // "Chinese (Taiwan)"]

// 한국어로 언어 표시 이름을 가져옵니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

@hochan222 en-US 원본은 중국번체이고, 예제코드이기 때문에 평소라면 즉각 수정 요청을 드리겠지만 이 문서는 다름이 아니라 Intl 이기 때문에 유지해도 나쁘지 않겠다 생각이 드는군요. 의견 부탁드려도 될까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

저도 처음엔 다른 번역들과 마찬가지로 중국어 번체로 번역을 진행했지만, 저희가 하는 것이 단순한 번역이 아닌 "지역화"가 목적인 만큼, 원문 예제의 형식에서 벗어나지 않는 범위에서 예제 수정을 부분적으로 허용해도 괜찮지 않을까 생각해봤습니다. 의도적으로 기존 규칙에서 벗어나 작업을 진행한 점은 저의 잘못이 맞지만, 지역화의 관점에서 봤을때 @wisedog 님은 어떻게 생각하시나요?

Copy link
Contributor

Choose a reason for hiding this comment

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

잘못이라뇨 가당치 않습니다. 저도 원래같으면 예제의 코멘트 외에 코드 수정을 하셨다면 바로 수정요청드렸겠지만, 이번엔 Intl 부분이라서 @SnowMarble 님의 기여 방식이 꽤 괜찮다고 생각해서 호찬님 의견을 물어본거거든요 : > 저도 이 부분을 생각지도 못했는데 좋은 제안 정말 감사합니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

@hochan222 님이 좀 바쁘신것 같습니다. 다른 부분은 다 오케이인데 이 부분만은 호찬님 의견이 중요할걸로 보여서 @SnowMarble 님 잠시만 더 기다려주시겠어요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 전 괜찮습니다:)

Copy link
Contributor

Choose a reason for hiding this comment

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

@hochan222 의견을 구합니다!

Copy link
Member

@hochan222 hochan222 Jul 30, 2023

Choose a reason for hiding this comment

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

@SnowMarble 님, @wisedog 님 안녕하세요. 답변이 늦어져서 정말 죄송합니다.

하는 것이 단순한 번역이 아닌 "지역화"가 목적인 만큼, 원문 예제의 형식에서 벗어나지 않는 범위에서 예제 수정을 부분적으로 허용해도 괜찮지 않을까 생각해봤습니다

좋은 방향이라고 생각합니다.

예시가 중국어 기준으로 된 이유는 중국 페이지 트래픽이 그당시 2위여서 그런듯합니다. 아래는 2020년(커밋 당시) 트래픽은 아니지만 참고차 첨부드립니다. (2021 web traffic compared to en-US)

// 2021 web traffic compared to en-US
en-US      58.9% (English)
zh-CN      8.9% (Chinese (China))
ja         6.6% (Japanese)
ru         6.3% (Russian)
es         6.1% (Spanish)
fr         4.3% (French)
pt-BR      3.4% (Portuguese (Brazil))
ko         2.5% (Korean)
de         1.4% (German)
zh-TW      1.0% (Chinese (Taiwan, Province of China))
pl         0.5% (Polish)

en-us 페이지에 너무 얽메이지 않고 좋은 지역화 방안이 있다면 반영해도 좋아보입니다.(대부분의 경우는 그대로 번역하는것이 좋아 보이지만, 현재 페이지는 SnowMarble님께서 제시해주신 방향이 좋아보이네용ㅎㅎ) 현재 경우는 문제 없어보이지만, 해당 내용 안내서에 추가해놓겠습니다.

감사합니다. 모두 좋은 주말 보내세요 :)

Copy link
Member

Choose a reason for hiding this comment

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

참고차, 최근 Translation guidelines for MDN translated content이 갱신되었습니다.

Localizing code blocks의 "If a code block must be rewritten to meet a locale's requirements, add a comment explaining why" 내용 추가되어서 참고차 댓글 남깁니다 :)


{{JSRef}}

**`Intl.DisplayNames`** 객체를 사용하면 언어, 지역 그리고 문자 체계 표시 이름을 일관되게 번역할 수 있습니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

script display names.문자 체계 표시 이름 이라 하셨는데 뭔가 딱 매칭되지가 않는데요. 부연설명 부탁드려도 될까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

script를 문자로 해석하는 것보단 문자 체계로 번역하는 것이 더 의미상 적정하다고 판단했습니다. 일본어 번역의 文字体系를 참고했습니다.

@SnowMarble SnowMarble requested a review from wisedog July 6, 2023 03:44
languageNames.of("en-US"); // "American English"
languageNames.of("zh-TW"); // "Chinese (Taiwan)"]

// 한국어로 언어 표시 이름을 가져옵니다.
Copy link
Member

@hochan222 hochan222 Jul 30, 2023

Choose a reason for hiding this comment

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

@SnowMarble 님, @wisedog 님 안녕하세요. 답변이 늦어져서 정말 죄송합니다.

하는 것이 단순한 번역이 아닌 "지역화"가 목적인 만큼, 원문 예제의 형식에서 벗어나지 않는 범위에서 예제 수정을 부분적으로 허용해도 괜찮지 않을까 생각해봤습니다

좋은 방향이라고 생각합니다.

예시가 중국어 기준으로 된 이유는 중국 페이지 트래픽이 그당시 2위여서 그런듯합니다. 아래는 2020년(커밋 당시) 트래픽은 아니지만 참고차 첨부드립니다. (2021 web traffic compared to en-US)

// 2021 web traffic compared to en-US
en-US      58.9% (English)
zh-CN      8.9% (Chinese (China))
ja         6.6% (Japanese)
ru         6.3% (Russian)
es         6.1% (Spanish)
fr         4.3% (French)
pt-BR      3.4% (Portuguese (Brazil))
ko         2.5% (Korean)
de         1.4% (German)
zh-TW      1.0% (Chinese (Taiwan, Province of China))
pl         0.5% (Polish)

en-us 페이지에 너무 얽메이지 않고 좋은 지역화 방안이 있다면 반영해도 좋아보입니다.(대부분의 경우는 그대로 번역하는것이 좋아 보이지만, 현재 페이지는 SnowMarble님께서 제시해주신 방향이 좋아보이네용ㅎㅎ) 현재 경우는 문제 없어보이지만, 해당 내용 안내서에 추가해놓겠습니다.

감사합니다. 모두 좋은 주말 보내세요 :)

@wisedog
Copy link
Contributor

wisedog commented Aug 1, 2023

@SnowMarble 멋진 PR 이었습니다. 늘 감사드립니다. 👍

@wisedog wisedog merged commit ec032ef into mdn:main Aug 1, 2023
@SnowMarble SnowMarble deleted the ko/intl.displayNames branch August 1, 2023 14:09
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.

3 participants