Skip to content

📝 docs: README 添加项目 Logo(透明背景) - #261

Merged
JunLang-7 merged 4 commits into
mainfrom
docs/add-readme-logo
Aug 14, 2026
Merged

📝 docs: README 添加项目 Logo(透明背景)#261
JunLang-7 merged 4 commits into
mainfrom
docs/add-readme-logo

Conversation

@ZhaoXingPeng

@ZhaoXingPeng ZhaoXingPeng commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

结论

README 首页添加透明背景的项目 Logo(assets/logo.png),与标题"VoiceLife 声活"并排居中展示(logo 在标题左侧)。希望 Reviewer 重点判断 logo 展示位置与抠图效果是否符合预期。

Refs #260

变更

  • 新增 assets/logo.png:由设计稿(1254×1254 PNG)用边缘连通 flood-fill 抠出透明背景,主体(蓝色图标 + 文字 + 底部装饰线)与内部白色细节完整保留。
  • README.md:在顶部 <div align="center"> 内,logo(宽 140px,align="middle")与 <h1>VoiceLife 声活</h1> 并排居中,logo 位于标题左侧。

明确未包含:

  • 未改动任何代码、构建配置或其他文档。
  • 未提交凭据、构建产物或用户本地未跟踪文件。

架构与兼容

不涉及。纯文档展示改动,无 Port、Profile、数据模型、协议或依赖变化。

验证

  • 视觉验证:本地合成深色背景确认背景已透明、主体完整(蓝色图标 + 文字均不透明)
  • ./scripts/run_pre_submit_checks.sh(不适用:纯 Markdown + 图片改动,不涉及代码/构建;已确认 assets/logo.png 未被 .gitignore 忽略,*.bin 类构建产物未纳入提交)
  • 远端 CI 的工作流、格式、主机测试等(README 与图片改动不触发构建类检查,等待 CI 结果确认)

证据:

  • 提交内容仅 README.md(+4 行)与 assets/logo.png(新增)。
  • 抠图验证:背景像素 alpha=0(如 (254,254,254)→alpha 0),主体 alpha=255,主体内部白色间隙保留不透明。

TDD 记录

不涉及。行为无变化,无代码逻辑。

风险与回退

  • 无产品行为风险。若对 logo 视觉效果不满意,可直接 revert 本 PR(或替换 assets/logo.png 重新生成)。

Review 清单

  • Issue 的验收标准已经逐项回应
  • 没有提交凭据、设备备份、用户隐私或构建产物
  • 新增/迁移的第三方代码记录了上游 commit 和许可(无新增第三方代码)
  • 文档和注释与实际行为一致
  • AI 生成内容已经由作者理解、测试并承担责任

@fennoai fennoai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found in this review. The README image reference, accessibility text, sizing markup, and transparent PNG asset are consistent with the documentation-only change.

Increased logo width from 88 to 120 pixels.

@JunLang-7 JunLang-7 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Good

@JunLang-7
JunLang-7 merged commit 69fb865 into main Aug 14, 2026
8 of 9 checks passed
@ZhaoXingPeng
ZhaoXingPeng deleted the docs/add-readme-logo branch August 14, 2026 05:25
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