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

[zh-cn]: update the translation of HTML <track> element #24499

Merged
merged 13 commits into from
Nov 21, 2024
Merged

[zh-cn]: update the translation of HTML <track> element #24499

merged 13 commits into from
Nov 21, 2024

Conversation

T34-active
Copy link
Contributor

@T34-active T34-active requested a review from a team as a code owner November 13, 2024 04:03
@T34-active T34-active requested review from irvin and removed request for a team November 13, 2024 04:03
@github-actions github-actions bot added the l10n-zh Issues related to Chinese content. label Nov 13, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 13, 2024 •
edited
Loading

Preview URLs

Flaws (12)

URL: /zh-CN/docs/Web/HTML/Element/track
Title: <track>：嵌入文本轨元素
Flaw count: 12

  • macros:
    • /zh-CN/docs/Web/HTML/Attributes/dirname does not exist but fell back to /en-US/docs/Web/HTML/Attributes/dirname
    • /zh-CN/docs/Web/HTML/Attributes/elementtiming does not exist but fell back to /en-US/docs/Web/HTML/Attributes/elementtiming
    • /zh-CN/docs/Web/HTML/Attributes/minlength does not exist but fell back to /en-US/docs/Web/HTML/Attributes/minlength
    • /zh-CN/docs/Web/HTML/Attributes/step does not exist but fell back to /en-US/docs/Web/HTML/Attributes/step
    • /zh-CN/docs/Web/API/HTMLTrackElement/track does not exist
    • and 7 more flaws omitted
External URLs (2)

URL: /zh-CN/docs/Web/HTML/Element/track
Title: <track>：嵌入文本轨元素

(comment last updated: 2024-11-21 09:02:12)

T34-active
T34-active commented Nov 13, 2024
View reviewed changes
Copy link
Contributor Author

@T34-active T34-active left a comment

Choose a reason for hiding this comment

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

fix format

files/zh-cn/web/html/element/track/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Outdated Show resolved Hide resolved
yin1999
yin1999 reviewed Nov 15, 2024
View reviewed changes
Copy link
Member

@yin1999 yin1999 left a comment

Choose a reason for hiding this comment

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

麻烦检查一下预览页面（有格式错误），有些词汇的翻译还得再注意一下（比如，我们不翻译“Web”一词）。请全文检查一遍，我再继续审查

T34-active
T34-active commented Nov 15, 2024
View reviewed changes
files/zh-cn/web/html/element/track/index.md Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Show resolved Hide resolved
@T34-active T34-active requested a review from yin1999 November 15, 2024 06:47
@yin1999
Copy link
Member

yin1999 commented Nov 16, 2024

不是，你没懂我的意思。定义列表的渲染有错误，你得检查一下格式。我希望这种错误能自己排查出来。

image

注：关于定义列表的格式，请参阅如何使用 Markdown 来撰写文档

T34-active and others added 4 commits November 18, 2024 09:38
@T34-active
Update index.md
7d515ab
@T34-active
Update index.md
145a9d1
@T34-active
Update index.md
b895129
@T34-active @github-actions
Apply suggestions from code review
5270406 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@T34-active @github-actions
Apply suggestions from code review
a698da8 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@T34-active
Copy link
Contributor Author

不是，你没懂我的意思。定义列表的渲染有错误，你得检查一下格式。我希望这种错误能自己排查出来。

image

注：关于定义列表的格式，请参阅如何使用 Markdown 来撰写文档

这个我改完之后还是会有这个问题，有什么页面可以参考一下的吗

yin1999
yin1999 requested changes Nov 18, 2024
View reviewed changes
files/zh-cn/web/html/element/track/index.md Outdated
Comment on lines 41 to 51
- `chapters`

- 视频内容的文本描述。
- 适用于失明用户或者当视频不可见的场景。
- 章节标题用于用户浏览媒体资源时使用。

- `chapters`
- `metadata`

- 章节标题用于用户浏览媒体资源的时候。
- 由脚本元素使用的轨道。对用户不可见。

- `metadata`
- `label`

- 脚本使用的 track。对用户不可见。
- : 用户可读的文本 track 标题，浏览器在列出可用文本 track 时使用。
Copy link
Member

Choose a reason for hiding this comment

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

看 diff 对比一下，这里的缩进被你破坏了。

@T34-active T34-active marked this pull request as draft November 18, 2024 07:23
@T34-active T34-active requested a review from yin1999 November 20, 2024 02:51
@T34-active T34-active marked this pull request as ready for review November 20, 2024 02:51
yin1999
yin1999 reviewed Nov 20, 2024
View reviewed changes
files/zh-cn/web/html/element/track/index.md Outdated
Comment on lines 10 to 12
**`<track>`** [HTML](/zh-CN/docs/Web/HTML) 元素作为媒体元素 {{HTMLElement("audio")}} 和 {{HTMLElement("video")}} 的子元素使用。每个文本 track 元素允许你指定一个定时文本 track（或基于时间的数据），可以与媒体元素并行显示，例如在视频上叠加字幕或隐藏式字幕，或与音频 track 一起显示。

一个`media` 元素的任意两个 `track` 子元素不能有相同的 `kind`, `srclang`, 和 `label`属性。
可以为一个 media 元素指定多个 track，包含不同种类的定时文本数据，或为不同区域翻译的定时文本数据。使用的数据将是已设置为默认的 track，或者基于用户偏好的类型和语言翻译。
Copy link
Member

Choose a reason for hiding this comment

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

这里为啥不翻译“track”和“media”，可以说一下是考虑到哪一点了吗

Copy link
Contributor Author

Choose a reason for hiding this comment

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

我的我的，我之前改错了。

T34-active
T34-active commented Nov 20, 2024
View reviewed changes
files/zh-cn/web/html/element/track/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/html/element/track/index.md Outdated Show resolved Hide resolved
yin1999
yin1999 approved these changes Nov 21, 2024
View reviewed changes
Copy link
Member

@yin1999 yin1999 left a comment

Choose a reason for hiding this comment

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

根据 #23060 修正了“cue”一词的翻译。感谢 @T34-active

@yin1999 yin1999 merged commit 186213a into mdn:main Nov 21, 2024
7 checks passed
@T34-active T34-active deleted the html_track_20241107 branch November 21, 2024 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

@yin1999 yin1999 yin1999 approved these changes

@irvin irvin Awaiting requested review from irvin irvin is a code owner automatically assigned from mdn/yari-content-zh

Assignees
No one assigned
Labels
l10n-zh Issues related to Chinese content.
Projects
MDN translated-content l10n-zh
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@T34-active @yin1999