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

[zh-cn]: create docs for HTMLEmbedElement #23080

Merged
merged 3 commits into from
Aug 24, 2024
Merged

Conversation

fuchunhui
Copy link
Contributor

Description

Motivation

Additional details

Related issues and pull requests

@fuchunhui fuchunhui requested a review from a team as a code owner August 15, 2024 12:45
@fuchunhui fuchunhui requested review from yin1999 and removed request for a team August 15, 2024 12:45
@github-actions github-actions bot added the l10n-zh Issues related to Chinese content. label Aug 15, 2024
Copy link
Contributor

github-actions bot commented Aug 15, 2024

Preview URLs

Flaws (9)

Note! 1 document with no flaws that don't need to be listed. 🎉

URL: /zh-CN/docs/Web/API/HTMLEmbedElement
Title: HTMLEmbedElement
Flaw count: 3

  • macros:
    • /zh-CN/docs/Web/API/HTMLEmbedElement/align does not exist
    • /zh-CN/docs/Web/API/HTMLEmbedElement/name does not exist
    • /zh-CN/docs/Web/API/HTMLEmbedElement/type does not exist

URL: /zh-CN/docs/Web/API/HTMLEmbedElement/height
Title: HTMLEmbedElement:height 属性
Flaw count: 3

  • macros:
    • /zh-CN/docs/Web/API/HTMLImageElement/height does not exist but fell back to /en-US/docs/Web/API/HTMLImageElement/height
    • /zh-CN/docs/Web/API/HTMLSourceElement/height does not exist but fell back to /en-US/docs/Web/API/HTMLSourceElement/height
    • /zh-CN/docs/Web/API/HTMLVideoElement/height does not exist but fell back to /en-US/docs/Web/API/HTMLVideoElement/height

URL: /zh-CN/docs/Web/API/HTMLEmbedElement/width
Title: HTMLEmbedElement:width 属性
Flaw count: 3

  • macros:
    • /zh-CN/docs/Web/API/HTMLImageElement/width does not exist but fell back to /en-US/docs/Web/API/HTMLImageElement/width
    • /zh-CN/docs/Web/API/HTMLSourceElement/width does not exist but fell back to /en-US/docs/Web/API/HTMLSourceElement/width
    • /zh-CN/docs/Web/API/HTMLVideoElement/width does not exist but fell back to /en-US/docs/Web/API/HTMLVideoElement/width

(comment last updated: 2024-08-24 00:41:55)

files/zh-cn/web/api/htmlembedelement/height/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/api/htmlembedelement/height/index.md Outdated Show resolved Hide resolved

{{APIRef("HTML DOM")}}

**`HTMLEmbedElement`** 接口提供特定的属性(除了常规 {{domxref("HTMLElement")}} 接口之外,它还可以通过继承来使用),用于操作 {{HTMLElement("embed")}} 元素。
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
**`HTMLEmbedElement`** 接口提供特定的属性(除了常规 {{domxref("HTMLElement")}} 接口之外,它还可以通过继承来使用),用于操作 {{HTMLElement("embed")}} 元素
**`HTMLEmbedElement`** 接口提供了用于操作 {{HTMLElement("embed")}} 元素的特殊属性(除了常规 {{domxref("HTMLElement")}} 接口之外,它还可以通过继承来使用)

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
Member

Choose a reason for hiding this comment

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

我觉得这句话的核心在于,此接口提供的特定属性/方法的作用,因此,我提供了一个把额外的说明后置的写法。可以参考我们在 #20679 中的讨论。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这几天详细看了下20679 的内容,但最后大家并没有给出一个更规范合理的表述方式,所以也很纠结到底选择哪种方式。我顺路问了下 AI,它是这样解释的,我觉得有些道理。

image

你的建议,比较合理。

Copy link
Member

Choose a reason for hiding this comment

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

请更新一下其他的几个 PR

files/zh-cn/web/api/htmlembedelement/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/api/htmlembedelement/src/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/api/htmlembedelement/width/index.md Outdated Show resolved Hide resolved
files/zh-cn/web/api/htmlembedelement/width/index.md Outdated Show resolved Hide resolved
@yin1999
Copy link
Member

yin1999 commented Aug 24, 2024

感谢 🙏

@yin1999 yin1999 merged commit 706cf99 into mdn:main Aug 24, 2024
7 checks passed
@fuchunhui
Copy link
Contributor Author

感谢 🙏

多谢帮忙更新,这几天有事没来得及处理。

@fuchunhui fuchunhui deleted the HTMLEmbedElement branch August 28, 2024 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-zh Issues related to Chinese content.
Projects
Development

Successfully merging this pull request may close these issues.

2 participants