fix: 标题预览仅在Step4显示,Step5不再显示标题叠加层 #1392

Merged

auto-approve-bot merged 1 commits from fix/title-preview-step4-only into develop 2026-08-16 18:41:12 +08:00

Conversation 4 Commits 1 Files Changed 1

+1 -1

xiaoxia commented 2026-08-16 18:31:30 +08:00

Owner

Copy Link

Copy Source

问题

用户在 Step5（生成预览视频）页面看到标题 Canvas 叠加层和右侧面板的预览视频重复，显得拥挤。

修复

• showTitlePreview 条件从 currentStep >= 4 && currentStep <= 5 改为 currentStep === 4
• Step5 预览视频页面只展示预览视频本身，不再叠加标题 Canvas
• 标题预览只在 Step4（选择标题）页面显示

验证

• tsc --noEmit 零错误
• prettier 格式通过

## 问题 用户在 Step5（生成预览视频）页面看到标题 Canvas 叠加层和右侧面板的预览视频重复，显得拥挤。 ## 修复 - `showTitlePreview` 条件从 `currentStep >= 4 && currentStep <= 5` 改为 `currentStep === 4` - Step5 预览视频页面只展示预览视频本身，不再叠加标题 Canvas - 标题预览只在 Step4（选择标题）页面显示 ## 验证 - [x] tsc --noEmit 零错误 - [x] prettier 格式通过

xiaoxia added the bugfrontend labels 2026-08-16 18:31:30 +08:00

xiaoxia added 1 commit 2026-08-16 18:31:30 +08:00

fix: 标题预览仅在Step4显示,Step5不再显示标题叠加层 ... 91165a4a34

- showTitlePreview 条件从 currentStep>=4&&<=5 改为 currentStep===4
- Step5预览视频页面只展示预览视频本身,不再叠加标题Canvas
- 避免与右侧面板预览视频重复导致视觉拥挤

auto-approve-bot commented 2026-08-16 18:32:56 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

• 整体评价：有建议
• 建议级问题数量：1 个

🔴 阻塞级问题（必须修复）

无

💡 改进建议（不阻塞合并）

1. [apps/web/src/pages/generate/GeneratePage.tsx:268] 业务逻辑变更确认
   • 具体内容：代码将 showTitlePreview 的判断条件从 currentStep >= 4 && currentStep <= 5（即步骤 4 和 5 均显示）修改为 currentStep === 4（仅步骤 4 显示）。考虑到上下文中存在 step5Preview 相关逻辑，请确认此变更是符合产品预期的需求调整（即步骤 5 不再需要显示标题预览），而非遗漏了对步骤 5 的处理。

✅ 良好实践

• 逻辑简化：将范围判断简化为相等性判断，在确认需求正确的前提下，提高了代码的可读性。

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-08-16 10:32:56 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：有建议 - 建议级问题数量：1 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[apps/web/src/pages/generate/GeneratePage.tsx:268] 业务逻辑变更确认** - 具体内容：代码将 `showTitlePreview` 的判断条件从 `currentStep >= 4 && currentStep <= 5`（即步骤 4 和 5 均显示）修改为 `currentStep === 4`（仅步骤 4 显示）。考虑到上下文中存在 `step5Preview` 相关逻辑，请确认此变更是符合产品预期的需求调整（即步骤 5 不再需要显示标题预览），而非遗漏了对步骤 5 的处理。 ### ✅ 良好实践 - 逻辑简化：将范围判断简化为相等性判断，在确认需求正确的前提下，提高了代码的可读性。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-16 10:32:56 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

gitea-actions bot commented 2026-08-16 18:34:00 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1392 | | 预览链接 | [https://pr-1392.preview.xiaoxiajianji.com](https://pr-1392.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。

auto-approve-bot approved these changes 2026-08-16 18:34:26 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-08-16 18:34:26 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia referenced this issue from a commit 2026-08-16 18:38:30 +08:00

fix: 标题预览仅在Step4显示,Step5不再显示标题叠加层 (#1392)

auto-approve-bot merged commit e96be1771a into develop 2026-08-16 18:41:12 +08:00

auto-approve-bot deleted branch fix/title-preview-step4-only 2026-08-16 18:41:12 +08:00

Sign in to join this conversation.

Reviewers

No Reviewers

auto-approve-bot

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label bug frontend

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1392