fix: PreviewVideoPanel仅在Step5+(预览页)显示,标题页不再显示预览视频 #1395

Merged
auto-approve-bot merged 1 commits from fix/preview-panel-step5-only into develop 2026-08-16 21:06:15 +08:00
Owner

问题

Step4(选择标题)页面右侧显示了预览视频面板,显得冗余。预览视频面板应该只在 Step5(预览)及之后的步骤显示。

修复

  • PreviewVideoPanel 渲染条件:currentStep >= 4currentStep >= 5
  • 标题 Canvas 叠加条件:currentStep === 4currentStep === 5

效果

步骤 修改前 修改后
Step4(标题) 标题设置 + 右侧预览视频面板 仅标题设置表单,右侧无面板
Step5(预览) 预览视频(无标题叠加) 预览视频 + 标题 Canvas 叠加
Step6+(封面/确认) 预览视频(无标题叠加) 预览视频(无标题叠加)

验证

  • tsc --noEmit 零错误
  • prettier 格式通过
## 问题 Step4(选择标题)页面右侧显示了预览视频面板,显得冗余。预览视频面板应该只在 Step5(预览)及之后的步骤显示。 ## 修复 - PreviewVideoPanel 渲染条件:`currentStep >= 4` → `currentStep >= 5` - 标题 Canvas 叠加条件:`currentStep === 4` → `currentStep === 5` ## 效果 | 步骤 | 修改前 | 修改后 | |------|--------|--------| | Step4(标题) | 标题设置 + 右侧预览视频面板 | 仅标题设置表单,右侧无面板 | | Step5(预览) | 预览视频(无标题叠加) | 预览视频 + 标题 Canvas 叠加 | | Step6+(封面/确认) | 预览视频(无标题叠加) | 预览视频(无标题叠加) | ## 验证 - [x] tsc --noEmit 零错误 - [x] prettier 格式通过
xiaoxia added the bugfrontend labels 2026-08-16 20:50:36 +08:00
xiaoxia added 1 commit 2026-08-16 20:50:36 +08:00
fix: PreviewVideoPanel仅在Step5+(预览页)显示,Step4(标题页)不再显示
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
CI/CD Pipeline / Build Staging API 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 29s
CI/CD Pipeline / Unit Tests (pull_request) Has been skipped
CI/CD Pipeline / PR Build Worker Image (pull_request) Has been skipped
CI/CD Pipeline / PR Build API Image (pull_request) Has been skipped
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m3s
AI Code Review / AI Code Review (pull_request) Successful in 1m20s
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m39s
CI/CD Pipeline / Frontend Unit Tests (pull_request) Successful in 1m16s
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 2m6s
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 2m16s
Preview Deploy / Deploy Preview Environment (pull_request) Successful in 2m50s
PR Automation / Auto Approve on CI Green (pull_request) Successful in 2m54s
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 2m44s
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 6m57s
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 / Canary Release to Production (pull_request) Has been skipped
CI/CD Pipeline / Deploy 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 7s
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 38s
ACR Cleanup / ACR Image Cleanup (pull_request_target) Successful in 57s
34a616fa18
- 渲染条件从 currentStep>=4 改为 currentStep>=5
- 标题Canvas叠加条件从 currentStep===4 改为 currentStep===5
- 效果: Step4标题页只有标题设置表单,右侧无预览视频面板
- 效果: Step5预览页显示预览视频+标题叠加
- 效果: Step6+仅显示预览视频(无标题叠加)
Collaborator

【阻塞级判定】

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

📊 审查概览

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

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

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

良好实践

  1. [apps/web/src/pages/generate/GeneratePage.tsx] 注释与逻辑同步更新
    • 具体内容:在修改组件渲染条件(currentStep >= 5)和属性传值(showTitlePreview)的同时,准确更新了上方的注释说明,确保代码逻辑与文档注释保持一致,便于团队理解业务流程变更。

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

### 【阻塞级判定】 - 是否存在阻塞级问题:否 - 阻塞级问题数量:0 个 ### 📊 审查概览 - 整体评价:通过 - 建议级问题数量:0 个 ### 🔴 阻塞级问题(必须修复) 无 ### 💡 改进建议(不阻塞合并) 无 ### ✅ 良好实践 1. **[apps/web/src/pages/generate/GeneratePage.tsx] 注释与逻辑同步更新** - 具体内容:在修改组件渲染条件(`currentStep >= 5`)和属性传值(`showTitlePreview`)的同时,准确更新了上方的注释说明,确保代码逻辑与文档注释保持一致,便于团队理解业务流程变更。 --- <sub>🤖 由 AI 代码审查机器人自动生成 | 2026-08-16 12:51:51 | 模型: </sub> <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

🚀 预览环境已部署

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

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

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

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

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

CI全绿,自动审批通过。

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

CI全绿,自动审批通过。

CI全绿,自动审批通过。
auto-approve-bot merged commit 8b0572362e into develop 2026-08-16 21:06:15 +08:00
auto-approve-bot deleted branch fix/preview-panel-step5-only 2026-08-16 21:06:15 +08:00

🗑️ 预览环境已清理

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

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

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