fix: 预览视频标题叠加 — Step4和Step5都显示标题Canvas #1390

Merged
auto-approve-bot merged 2 commits from fix/preview-title-overlay into develop 2026-08-16 17:21:00 +08:00
Owner

问题

  1. 预览视频(Step5)没有显示标题文字 — 因为 showTitlePreview 仅在 currentStep === 4 时为 true
  2. PreviewVideoPanel 空状态文案引用了旧的步骤编号

修复

  1. GeneratePage.tsx: showTitlePreview={currentStep >= 4 && currentStep <= 5} — Step4 和 Step5 都在预览画面上叠加标题 Canvas
  2. PreviewVideoPanel.tsx: 标题逻辑优化 — 有生成视频时显示"预览视频"标题,空状态文案去掉硬编码步骤号

步骤顺序说明

代码中步骤顺序已是正确的(PR #1377 已合入):

  • Step 4 = 选择标题
  • Step 5 = 生成预览

staging 前端未更新是部署问题(web 镜像未被拉取),与此 PR 无关。

## 问题 1. 预览视频(Step5)没有显示标题文字 — 因为 `showTitlePreview` 仅在 `currentStep === 4` 时为 true 2. PreviewVideoPanel 空状态文案引用了旧的步骤编号 ## 修复 1. `GeneratePage.tsx`: `showTitlePreview={currentStep >= 4 && currentStep <= 5}` — Step4 和 Step5 都在预览画面上叠加标题 Canvas 2. `PreviewVideoPanel.tsx`: 标题逻辑优化 — 有生成视频时显示"预览视频"标题,空状态文案去掉硬编码步骤号 ## 步骤顺序说明 代码中步骤顺序已是正确的(PR #1377 已合入): - Step 4 = 选择标题 - Step 5 = 生成预览 staging 前端未更新是部署问题(web 镜像未被拉取),与此 PR 无关。
xiaoxia added 2 commits 2026-08-16 17:14:55 +08:00
fix: 更新预览面板标题叠加逻辑和空状态文案
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 43s
CI/CD Pipeline / Unit Tests (pull_request) Has been skipped
CI/CD Pipeline / PR Build API Image (pull_request) Has been skipped
CI/CD Pipeline / PR Build Worker Image (pull_request) Has been skipped
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m48s
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 1m48s
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m48s
CI/CD Pipeline / Frontend Unit Tests (pull_request) Successful in 1m10s
Preview Deploy / Deploy Preview Environment (pull_request) Successful in 2m27s
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 2m7s
PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m0s
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 2m29s
AI Code Review / AI Code Review (pull_request) Successful in 3m26s
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 4m55s
CI/CD Pipeline / Integration Tests (pull_request) Has been skipped
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
CI/CD Pipeline / CI Gate (pull_request) Successful in 6s
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 35s
ACR Cleanup / ACR Image Cleanup (pull_request_target) Successful in 39s
8f85ec254f

🚀 预览环境已部署

项目 详情
PR号 #1390
预览链接 https://pr-1390.preview.xiaoxiajianji.com
API环境 staging

💡 预览环境使用 staging API 数据,请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后,预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1390 | | 预览链接 | [https://pr-1390.preview.xiaoxiajianji.com](https://pr-1390.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据,请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后,预览环境会自动清理。
auto-approve-bot approved these changes 2026-08-16 17:17:55 +08:00
auto-approve-bot left a comment
Collaborator

CI全绿,自动审批通过。

CI全绿,自动审批通过。
auto-approve-bot approved these changes 2026-08-16 17:17:56 +08:00
auto-approve-bot left a comment
Collaborator

CI全绿,自动审批通过。

CI全绿,自动审批通过。
Collaborator

【阻塞级判定】

  • 是否存在阻塞级问题:否
  • 阻塞级问题数量:0 个

📊 审查概览

  • 整体评价:通过
  • 建议级问题数量:0 个

🔴 阻塞级问题(必须修复)

💡 改进建议(不阻塞合并)

良好实践

  • 逻辑调整清晰:针对 currentStep 的范围调整(>=4 && <=5)与标题显示逻辑(showTitlePreview && !hasPreview)配合默契,准确区分了“步骤4标题编辑”与“步骤5生成结果”两种状态下的面板展示。
  • 文案优化:将“在第 3 步生成预览后在此查看”修改为“在左侧生成预览后在此查看”,提升了用户指引的通用性和准确性。

🤖 由 AI 代码审查机器人自动生成 | 2026-08-16 09:18:21 | 模型:

### 【阻塞级判定】 - 是否存在阻塞级问题:否 - 阻塞级问题数量:0 个 ### 📊 审查概览 - 整体评价:通过 - 建议级问题数量:0 个 ### 🔴 阻塞级问题(必须修复) 无 ### 💡 改进建议(不阻塞合并) 无 ### ✅ 良好实践 - 逻辑调整清晰:针对 `currentStep` 的范围调整(`>=4 && <=5`)与标题显示逻辑(`showTitlePreview && !hasPreview`)配合默契,准确区分了“步骤4标题编辑”与“步骤5生成结果”两种状态下的面板展示。 - 文案优化:将“在第 3 步生成预览后在此查看”修改为“在左侧生成预览后在此查看”,提升了用户指引的通用性和准确性。 --- <sub>🤖 由 AI 代码审查机器人自动生成 | 2026-08-16 09:18:21 | 模型: </sub> <!-- AI_CODE_REVIEW_AUTO_COMMENT -->
auto-approve-bot merged commit 9eb1c78d5e into develop 2026-08-16 17:21:00 +08:00
auto-approve-bot deleted branch fix/preview-title-overlay 2026-08-16 17:21:00 +08:00

🗑️ 预览环境已清理

PR #1390 已关闭或合并,对应的预览环境已被清理。

如有需要,可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #1390 已关闭或合并,对应的预览环境已被清理。 > 如有需要,可以重新打开 PR 来重新生成预览环境。
Sign in to join this conversation.