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] api/htmlelement/tabindex 신규 번역 #23600

Merged
merged 7 commits into from
Sep 29, 2024

Conversation

rosaceaee
Copy link
Contributor

안녕하세요. tabindex 페이지 신규 번역하였습니다.

@rosaceaee rosaceaee requested a review from a team as a code owner September 16, 2024 07:25
@rosaceaee rosaceaee requested review from hochan222 and removed request for a team September 16, 2024 07:25
@github-actions github-actions bot added the l10n-ko Issues related to Korean content. label Sep 16, 2024
rosaceaee and others added 2 commits September 16, 2024 20:38
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

@rosaceaee 님 안녕하세요.

몇가지 리뷰사항 하기엔 남겨두었습니다. 확인 부탁드립니다.

고생 정말 많으셨습니다. 좋은 하루 보내세요 :)


**`tabIndex`** 는 현재 요소의 탭 순서를 나타내는 {{DOMxRef("HTMLElement")}} 인터페이스입니다.

탭 순서의 흐름
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
순서의 흐름
순서는 다음과 같습니다.

문장이 더 명확하게 전달될 수 있도록 제안드립니다.


탭 순서의 흐름

1. `tabIndex`가 정수인 요소인 경우.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
1. `tabIndex`정수인 요소인 경우.
1. `tabIndex`양수인 요소인 경우.

Elements with a positive tabIndex.

Comment on lines 15 to 16
동일한 `tabIndex`값을 가진 요소는 차례대로 나타난 순서로 표시됩니다.
이 순서는 `tabIndex` 요소의 가장 낮은 값에서 가장 높은 값으로 진행됩니다.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
동일한 `tabIndex`값을 가진 요소는 차례대로 나타난 순서로 표시됩니다.
이 순서는 `tabIndex` 요소의 가장 낮은 값에서 가장 높은 값으로 진행됩니다.
동일한 `tabIndex` 값을 가진 요소들은 나타나는 순서대로 탐색되어야 합니다.
탐색은 가장 낮은 `tabIndex`에서 가장 높은 `tabIndex` 진행됩니다.

should be 와 Navigation의 의미가 더 잘 전달되도록 제안드립니다.

Elements that have identical tabIndex values should be navigated in the order they appear.
Navigation proceeds from the lowest tabIndex to the highest tabIndex.

1. `tabIndex`가 정수인 요소인 경우.
동일한 `tabIndex`값을 가진 요소는 차례대로 나타난 순서로 표시됩니다.
이 순서는 `tabIndex` 요소의 가장 낮은 값에서 가장 높은 값으로 진행됩니다.
2. `tabIndex` 속성을 지원하지 않거나 지원하지만 `tabIndex` 값이 `0`인 경우에는 순서대로 표시됩니다.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
2. `tabIndex` 속성을 지원하지 않거나 지원하지만 `tabIndex` 값이 `0`인 경우에는 순서대로 표시됩니다.
2. `tabIndex` 속성을 지원하지 않거나 지원하면서 `tabIndex``0`으로 지정한 요소들은 나타나는 순서대로 탐색됩니다.

[제안]

이 순서는 `tabIndex` 요소의 가장 낮은 값에서 가장 높은 값으로 진행됩니다.
2. `tabIndex` 속성을 지원하지 않거나 지원하지만 `tabIndex` 값이 `0`인 경우에는 순서대로 표시됩니다.

disabled 속성을 가진 요소는 탭 순서에 들어가지 않습니다.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
disabled 속성을 가진 요소는 탭 순서에 들어가지 않습니다.
비활성화된 요소들은 탭 순서에 참여하지 않습니다.

markdown code 문법이 아니여서 번역해도 좋을듯합니다. disabled보다는 비활성화의 의미가 더 가까운듯하여 리뷰드렸습니다.


disabled 속성을 가진 요소는 탭 순서에 들어가지 않습니다.

값이 순서대로 부여되지 않아도 되며 특정 값으로 시작되어도 됩니다. 음수인 경우라면 각 브라우저에서 매우 큰 값으로 정돈하여 나타냅니다.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
값이 순서대로 부여되지 않아도 되며 특정 값으로 시작되어도 됩니다. 음수인 경우라면 각 브라우저에서 매우 큰 값으로 정돈하여 나타냅니다.
값은 순차적일 필요가 없으며, 특정 값으로 시작할 필요도 없습니다. 값은 음수일 수도 있지만, 각 브라우저는 매우 큰 값을 잘라냅니다.

nor must를 고려한 번역 제안 드립니다!

Values don't need to be sequential, nor must they begin with any particular value. They
may even be negative, though each browser trims very large values.

b1.tabIndex = 1;
```

## 명세
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
## 명세
## 명세서

단어 안내서에 따라 명세서로 수정 부탁드립니다!

@hochan222 hochan222 self-requested a review September 22, 2024 09:14
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.

.

@rosaceaee
Copy link
Contributor Author

@hochan222 님 상세한 리뷰 감사합니다!
리뷰해주신 내용 반영하여 커밋하였습니다!

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

고생 많으셨습니다~! 좋은 하루 보내세요 :)

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.

@rosaceaee 님 안녕하세요.

현재 PR Test / tests (pull_request_target) Failing after 42s git action이 실패하고 있습니다.

image

현재 경로는 files/ko/web/api/htmlelement/tabIndex/index.md인데, files/ko/web/api/htmlelement/tabindex/index.md 경로가 올바른 경로라서 수정이 필요해보입니다. (tabIndex => tabindex)

Copy link
Contributor

Preview URLs

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.

상기 오류 제가 수정 후 반영했습니다. 좋은 하루 보내세요 :)

@hochan222 hochan222 merged commit b1bab72 into mdn:main Sep 29, 2024
7 checks passed
@rosaceaee
Copy link
Contributor Author

@hochan222 수정해주셔서 감사합니다! 제가 대소문자 구분하는걸 누락했네요 죄송합니다 ㅠㅠ

sora32127 pushed a commit to sora32127/translated-content that referenced this pull request Oct 6, 2024
* [ko] api/htmlelement/tabindex 신규 번역

* 린트에 맞추어 공백 제거

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update files/ko/web/api/htmlelement/tabIndex/index.md

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Tabindex 리뷰 내용 반영

* Update files/ko/web/api/htmlelement/tabIndex/index.md

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* fix: 잘못된 파일 경로 수정

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

* fix: 잘못된 파일 경로 수정

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

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: hochan Lee <[email protected]>
Co-authored-by: rosaceaee <[email protected]>
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