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: create notifications.NotificationOptions #25077

Merged
merged 8 commits into from
Dec 23, 2024
Merged

zh-CN: create notifications.NotificationOptions #25077

merged 8 commits into from
Dec 23, 2024

Conversation

PassionPenguin
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the l10n-zh Issues related to Chinese content. label Dec 15, 2024
@PassionPenguin PassionPenguin marked this pull request as ready for review December 15, 2024 13:23
@PassionPenguin PassionPenguin requested a review from a team as a code owner December 15, 2024 13:23
@PassionPenguin PassionPenguin requested review from JasonLamv-t and removed request for a team December 15, 2024 13:23
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Dec 15, 2024 •
edited
Loading

Preview URLs

Flaws (13)

URL: /zh-CN/docs/Mozilla/Add-ons/WebExtensions/API/notifications/NotificationOptions
Title: notifications.NotificationOptions
Flaw count: 13

  • macros:
    • /zh-CN/docs/Mozilla/Add-ons/WebExtensions/Background_scripts does not exist but fell back to /en-US/docs/Mozilla/Add-ons/WebExtensions/Background_scripts
    • /zh-CN/docs/Mozilla/Add-ons/WebExtensions/Working_with_files does not exist but fell back to /en-US/docs/Mozilla/Add-ons/WebExtensions/Working_with_files
    • /zh-CN/docs/Mozilla/Add-ons/WebExtensions/Differences_between_API_implementations does not exist but fell back to /en-US/docs/Mozilla/Add-ons/WebExtensions/Differences_between_API_implementations
    • /zh-CN/docs/Mozilla/Add-ons/WebExtensions/user_interface/Sidebars does not exist but fell back to /en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface/Sidebars
    • /zh-CN/docs/Mozilla/Add-ons/WebExtensions/user_interface/Context_menu_items does not exist but fell back to /en-US/docs/Mozilla/Add-ons/WebExtensions/user_interface/Context_menu_items
    • and 8 more flaws omitted
External URLs (1)

URL: /zh-CN/docs/Mozilla/Add-ons/WebExtensions/API/notifications/NotificationOptions
Title: notifications.NotificationOptions

(comment last updated: 2024-12-23 07:36:24)

yin1999
yin1999 reviewed Dec 19, 2024
View reviewed changes
files/zh-cn/mozilla/add-ons/webextensions/api/notifications/notificationoptions/index.md Outdated Show resolved Hide resolved
files/zh-cn/mozilla/add-ons/webextensions/api/notifications/notificationoptions/index.md Outdated Show resolved Hide resolved
files/zh-cn/mozilla/add-ons/webextensions/api/notifications/notificationoptions/index.md Outdated
- `type`
- : {{WebExtAPIRef("notifications.TemplateType")}}。你所希望的通知类型。根据你的选择，某些其他属性可能是必需的或者是不允许的。
- `message`
- : `string`，通知的正文内容。
Copy link
Member

Choose a reason for hiding this comment

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

类型后面的标点不统一，有逗号、有句号，还有冒号

files/zh-cn/mozilla/add-ons/webextensions/api/notifications/notificationoptions/index.md Outdated

- `imageUrl`

- : `string`，指向通知中显示的图像的 URL。URL 可以是：data URL、blob URL、http 或 https URL，或者是扩展内的文件的[相对 URL](/zh-CN/docs/Mozilla/Add-ons/WebExtensions/Chrome_incompatibilities#relative_urls)。使用 SVG 图像时，请确保图像包含高度和宽度属性，例如 `<svg width="96" height="96"…`。否则，图像可能无法显示。
Copy link
Member

Choose a reason for hiding this comment

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

英文文档引用的小节不存在，中文的也不存在（即使翻译片段值）

Copy link
Contributor Author

Choose a reason for hiding this comment

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

英文文档的broken url删漏了

@PassionPenguin PassionPenguin marked this pull request as draft December 21, 2024 13:22
@PassionPenguin PassionPenguin marked this pull request as ready for review December 23, 2024 07:34
@yin1999 yin1999 merged commit f50a053 into mdn:main Dec 23, 2024
7 checks passed
@PassionPenguin PassionPenguin deleted the notifications-3 branch December 23, 2024 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yin1999 yin1999 yin1999 approved these changes

@JasonLamv-t JasonLamv-t Awaiting requested review from JasonLamv-t JasonLamv-t 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
@PassionPenguin @yin1999