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.

Sign up for GitHub

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

Jump to bottom

[ko] logical properties - basic concepts 신규 번역 #21745

Merged
merged 6 commits into from
Sep 29, 2024
Merged

[ko] logical properties - basic concepts 신규 번역 #21745

merged 6 commits into from
Sep 29, 2024

Conversation

givvemee
Copy link
Contributor

Description

기존에 한국어로 존재하지 않던 문서 Basic concepts of logical properties and values를 번역합니다

Motivation

Additional details

대열 레이아웃과 쓰기 모드 에서는 flow layout 을 대열 레이아웃이라고 번역한 것을 뒤늦게 발견했는데요, 이 문서와 단어를 맞춰야 할지 고민입니다. 한자어가 오히려 어렵게 느껴질 것 같더라고요.

Related issues and pull requests

@givvemee givvemee requested a review from a team as a code owner June 19, 2024 01:29
@givvemee givvemee requested review from sunhpark42 and removed request for a team June 19, 2024 01:29
@github-actions github-actions bot added the l10n-ko Issues related to Korean content. label Jun 19, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jun 19, 2024 •
edited
Loading

Preview URLs

Flaws (7)

URL: /ko/docs/Web/CSS/CSS_logical_properties_and_values/Basic_concepts_of_logical_properties_and_values
Title: 논리적 속성과 값의 기본 개념
Flaw count: 7

  • macros:
    • /ko/docs/Glossary/physical_properties does not exist but fell back to /en-US/docs/Glossary/Physical_properties
    • /ko/docs/Glossary/flow_relative_values does not exist but fell back to /en-US/docs/Glossary/Flow_relative_values
    • /ko/docs/Glossary/physical_properties does not exist but fell back to /en-US/docs/Glossary/Physical_properties
    • /ko/docs/Web/CSS/align-self does not exist but fell back to /en-US/docs/Web/CSS/align-self
    • /ko/docs/Web/CSS/justify-self does not exist but fell back to /en-US/docs/Web/CSS/justify-self
    • and 2 more flaws omitted
External URLs (1)

URL: /ko/docs/Web/CSS/CSS_logical_properties_and_values/Basic_concepts_of_logical_properties_and_values
Title: 논리적 속성과 값의 기본 개념

(comment last updated: 2024-08-19 12:46:56)

sunhpark42
sunhpark42 requested changes Jun 30, 2024
View reviewed changes
Copy link
Contributor

@sunhpark42 sunhpark42 left a comment

Choose a reason for hiding this comment

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

안녕하세요 @givvemee 님, 번역 감사합니다. 내용 확인했습니다.
번역상 저로 flow layout이 대열 레이아웃으로 번역되는 경우보다는 다른 표현을 사용할 수 있다면 사용하는 것이 좋다고 생각이 들어서, 리뷰어 회의시에 논제로 꺼내보겠습니다. 의견 감사합니다:)

번역상 모호한 부분이 있어서 코멘트로 함께 남겨두었습니다. 확인 부탁드립니다.
감사합니다 :)

...s/css_logical_properties_and_values/basic_concepts_of_logical_properties_and_values/index.md Outdated Show resolved Hide resolved
...s/css_logical_properties_and_values/basic_concepts_of_logical_properties_and_values/index.md Outdated Show resolved Hide resolved
...s/css_logical_properties_and_values/basic_concepts_of_logical_properties_and_values/index.md Outdated Show resolved Hide resolved
@givvemee givvemee requested a review from sunhpark42 July 1, 2024 00:23
@givvemee
Copy link
Contributor Author

givvemee commented Jul 1, 2024

@sunhpark42 대열 레이아웃 이라는 표현은 이 문서 에서 가져온 것이긴 한데요, 저는 logical properties and values 내의 문서들을 번역하면서 flow-relative, flow-layout 은 모두 흐름~ 으로 번역했어요. 그 편이 더 자연스러운 것 같아서요. 대열이라는 한자어를 한 번 더 파악해야 하는 수고로움이 있을 것 같기도 했고요. 참고 부탁드립니다.

hochan222
hochan222 previously requested changes Jul 28, 2024
View reviewed changes
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.

이미지 파일들은 자동으로 경로가 지정되어서 yari에서 제공되고 있습니다. 이미지 파일들 모두 제거 부탁드립니다~!

@givvemee
Copy link
Contributor Author

@hochan222 수정되었어요! 감사합니다.

@givvemee givvemee requested a review from hochan222 July 30, 2024 00:45
@hochan222
Copy link
Member

hochan222 commented Jul 30, 2024 •
edited
Loading

@sunhpark42 님, @givvemee 님 안녕하세요.

개인적으로는 음차가 좋아보이나.. 현재 PR은 기존에 존재하는 다른 문서에서 대열 레이아웃으로 번역이 진행되어, 대열 레이아웃으로 번역하는것이 좋아 보입니다.

나머지 고민은 "음차 및 한국어 번역에 대한 기준 마련"에서 다루는것이 좋아 보입니다. 상기 링크에서 논의한 후에 수정하는건 어떨까요? 의견 부탁드립니다.

모두 좋은 하루 보내세요 :)

@sunhpark42
Copy link
Contributor

확인 및 전달 감사합니다 호찬님,
@givvemee 님 7월중에 회사 일정이 바빠, 확인이 너무 늦어져서 죄송합니다. ㅠㅠ
다른 부분확인 되어서, 호찬님 코멘트 확인 후 반영해주시면 머지하도록 하겠습니다. 기다려주셔서 감사합니다. 🙇‍♀️

@givvemee
Copy link
Contributor Author

givvemee commented Aug 5, 2024

@hochan222 이거 논의 중이지만 대열 레이아웃 으로 수정하여 일단 머지는 해두어야 할까요?

@sunhpark42
Copy link
Contributor

@givvemee 님 대열 레이아웃으로 작성 부탁드리며, 해당 논의 완료이후 수정으로 결정되면 해당 문구 일괄 수정하는 것이 좋을 것 같아, 수정부탁드립니다~!

@givvemee
Copy link
Contributor Author

@givvemee 님 대열 레이아웃으로 작성 부탁드리며, 해당 논의 완료이후 수정으로 결정되면 해당 문구 일괄 수정하는 것이 좋을 것 같아, 수정부탁드립니다~!

제가 너무 늦어버렸네요 ... ㅎㅎ. 수정 완료하여 커밋했습니다. 감사해요!

sunhpark42
sunhpark42 approved these changes Sep 29, 2024
View reviewed changes
Copy link
Contributor

@sunhpark42 sunhpark42 left a comment

Choose a reason for hiding this comment

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

안녕하세요 @givvemee 님 확인이 많이 늦었습니다. 죄송합니다 ㅠㅠ 수정사항 확인했습니다. 머지할게요! 항상 기여 감사합니다:)

@sunhpark42 sunhpark42 removed the request for review from hochan222 September 29, 2024 12:29
@sunhpark42 sunhpark42 merged commit 3c0c716 into mdn:main Sep 29, 2024
7 checks passed
sora32127 pushed a commit to sora32127/translated-content that referenced this pull request Oct 6, 2024
@givvemee @sunhpark42 @sora32127
[ko] logical properties - basic concepts 신규 번역 (mdn#21745)
dd97ea4 
* feat: [ko] logical properties - basic concepts 신규 번역

* Update files/ko/web/css/css_logical_properties_and_values/basic_concepts_of_logical_properties_and_values/index.md

Co-authored-by: sunny <[email protected]>

* [fix] smoother sentence

* [fix] 이미지 파일 삭제

* fix: words

---------

Co-authored-by: sunny <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sunhpark42 sunhpark42 sunhpark42 approved these changes

@hochan222 hochan222 hochan222 left review comments

Assignees
No one assigned
Labels
l10n-ko Issues related to Korean content.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@givvemee @hochan222 @sunhpark42