Skip to content

fix: add missing zh_HK help manual images - #530

Closed
GongHeng2017 wants to merge 1 commit into
linuxdeepin:release/eaglefrom
GongHeng2017:fix/bug-357679-zh-hk-missing-images
Closed

GongHeng2017 wants to merge 1 commit into
linuxdeepin:release/eaglefrom
GongHeng2017:fix/bug-357679-zh-hk-missing-images

Conversation

@GongHeng2017

Copy link
Copy Markdown
Contributor

问题

PMS 357679: zh_HK 帮助手册中 power_event.png 和 rightclick.png 图片缺失,导致渲染时图片显示异常。

根因

zh_HK/log-viewer.md 引用了 9 个图片文件,但 zh_HK/fig/ 目录仅有 7 个,缺少 power_event.png(第 143 行)和 rightclick.png(第 213 行)。

修复方案

将 power_event.png 和 rightclick.png 从 zh_CN/fig/ 复制到 zh_HK/fig/,补齐缺失的图片资源。零代码逻辑改动。

改动范围

  • 新增 application/assets/deepin-log-viewer/log-viewer/zh_HK/fig/power_event.png
  • 新增 application/assets/deepin-log-viewer/log-viewer/zh_HK/fig/rightclick.png

关联

zh_HK/log-viewer.md references power_event.png and rightclick.png,
but these files were missing from zh_HK/fig/, causing the help manual
to render broken images. Copy them from zh_CN/fig/ to fix the issue.

Fixes: PMS 357679
@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: GongHeng2017

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@deepin-ci-robot

Copy link
Copy Markdown

deepin pr auto review

🤖 AI 代码审查报告

总体评分: 100 分 (通过阈值: 70分)

Pass


📊 总体评价

项目 结果
审查结论 代码审查通过
评分详情 本次 PR 仅添加了 2 个 PNG 图片文件(繁体中文香港版帮助手册图片),无代码变更。所有文件均被过滤为静态资源,无代码需要分析。未发现任何安全问题。

🔍 详细分析

1. 语法逻辑 ✅

评价: 优秀 ✅ 通过

潜在问题:
✅ 未发现明显问题

建议: []


2. 代码质量 ✅

评价: 优秀 ✅ 通过

潜在问题:
✅ 未发现明显问题

建议: []


3. 代码性能 ✅

评价: 优秀 ✅ 通过

潜在问题:
✅ 未发现明显问题

建议: []


4. 代码安全 🔒

评价: 优秀 ✅ 通过

🔐 发现 0 个安全漏洞

安全漏洞详情:
✅ 未发现安全漏洞

建议: []


💡 改进建议代码示例

// 暂无代码示例

本报告由 AI 代码审查工具自动生成

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants