fix: 标题默认位置 fallback 改为 bottom，与前端 DEFAULT_TITLE_SETTINGS 对齐 #1691

Merged

xiaoxia merged 1 commits from fix/title-position-fallback-wrap-consistency into develop 2026-09-04 15:19:37 +08:00

Conversation 4 Commits 1 Files Changed 6

+69 -27

xiaoxia commented 2026-09-04 15:14:35 +08:00

Owner

Copy Link

Copy Source

改动内容

后端修改（3 处）

1. position_to_ass_alignment() fallback 从 8(top) 改为 2(bottom)

   • 文件：packages/domain/ass_subtitle_builder.py
   • 与前端 DEFAULT_TITLE_SETTINGS.position = "bottom" 对齐
   • docstring 同步更新

2. build_ass_content() title 默认 position 从 "top" 改为 "bottom"

   • 文件：packages/domain/ass_subtitle_builder.py
   • 当前端不传 position 时，后端默认走底部对齐

3. subtitle_generator.generate_ass_from_timeline() title 默认 position 同步

   • 文件：apps/worker/video_processing/subtitle_generator.py
   • title_cfg.get("position", "top") → title_cfg.get("position", "bottom")

关于 Task 5（换行一致性）

_wrap_title_text 已使用 available_width = video_width - margin_l - margin_r 作为固定宽度约束，与前端基于百分比的约束逻辑一致。换行计算使用原始 font_size（不做 1.35x 补偿），与 CSS 预览保持一致。

测试更新

• 更新 6 个测试文件中 fallback 断言（alignment 8→2）
• 新增 TestDefaultPositionBottom 测试类（3 个测试）
• 351 个标题/字幕相关测试全通过

关联

与前端工程师的 PR 配合使用，确保预览和最终生成视频标题位置一致。

## 改动内容 ### 后端修改（3 处） 1. **`position_to_ass_alignment()` fallback 从 8(top) 改为 2(bottom)** - 文件：`packages/domain/ass_subtitle_builder.py` - 与前端 `DEFAULT_TITLE_SETTINGS.position = "bottom"` 对齐 - docstring 同步更新 2. **`build_ass_content()` title 默认 position 从 `"top"` 改为 `"bottom"`** - 文件：`packages/domain/ass_subtitle_builder.py` - 当前端不传 position 时，后端默认走底部对齐 3. **`subtitle_generator.generate_ass_from_timeline()` title 默认 position 同步** - 文件：`apps/worker/video_processing/subtitle_generator.py` - `title_cfg.get("position", "top")` → `title_cfg.get("position", "bottom")` ### 关于 Task 5（换行一致性） `_wrap_title_text` 已使用 `available_width = video_width - margin_l - margin_r` 作为固定宽度约束，与前端基于百分比的约束逻辑一致。换行计算使用原始 `font_size`（不做 1.35x 补偿），与 CSS 预览保持一致。 ### 测试更新 - 更新 6 个测试文件中 fallback 断言（alignment 8→2） - 新增 `TestDefaultPositionBottom` 测试类（3 个测试） - **351 个标题/字幕相关测试全通过** ### 关联 与前端工程师的 PR 配合使用，确保预览和最终生成视频标题位置一致。

xiaoxia added 1 commit 2026-09-04 15:14:36 +08:00

fix: 标题默认位置 fallback 改为 bottom，与前端 DEFAULT_TITLE_SETTINGS 对齐 ... 15e429ffb1

- position_to_ass_alignment() fallback 从 8(top) 改为 2(bottom)
- build_ass_content() title 默认 position 从 'top' 改为 'bottom'
- subtitle_generator.generate_ass_from_timeline() title 默认 position 同步
- 更新 6 个测试文件中 fallback 断言（8→2）
- 新增 TestDefaultPositionBottom 测试类（3 个测试）
- 351 个标题/字幕相关测试全通过

auto-approve-bot commented 2026-09-04 15:15:20 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

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

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

无

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

无

✅ 良好实践

1. 测试覆盖全面：在修改默认对齐方式从 top 到 bottom 时，同步更新了所有相关单元测试（包括 unknown、empty string、None 等边界情况），并新增了 TestDefaultPositionBottom 测试类，确保了行为变更的一致性和可靠性。
2. 文档同步更新：修改了 packages/domain/ass_subtitle_builder.py 中 position_to_ass_alignment 函数的文档字符串，明确指出了新的默认值及其与前端配置的对齐关系，有助于后续维护。
3. 逻辑一致性：代码变更在 subtitle_generator.py 和 ass_subtitle_builder.py 之间保持了逻辑一致，统一将默认位置参数调整为 "bottom"，避免了潜在的配置冲突。

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-09-04 07:15:20 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 1. **测试覆盖全面**：在修改默认对齐方式从 `top` 到 `bottom` 时，同步更新了所有相关单元测试（包括 `unknown`、`empty string`、`None` 等边界情况），并新增了 `TestDefaultPositionBottom` 测试类，确保了行为变更的一致性和可靠性。 2. **文档同步更新**：修改了 `packages/domain/ass_subtitle_builder.py` 中 `position_to_ass_alignment` 函数的文档字符串，明确指出了新的默认值及其与前端配置的对齐关系，有助于后续维护。 3. **逻辑一致性**：代码变更在 `subtitle_generator.py` 和 `ass_subtitle_builder.py` 之间保持了逻辑一致，统一将默认位置参数调整为 `"bottom"`，避免了潜在的配置冲突。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-09-04 07:15:20 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

gitea-actions bot commented 2026-09-04 15:16:06 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia reviewed 2026-09-04 15:19:18 +08:00

xiaoxia left a comment

Author

Owner

Copy Link

Copy Source

LGTM ✅ CI 全绿，改动正确。

LGTM ✅ CI 全绿，改动正确。

xiaoxia merged commit e86f137c3d into develop 2026-09-04 15:19:37 +08:00

gitea-actions bot commented 2026-09-04 15:20:04 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1691 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

xiaoxia referenced this pull request 2026-09-04 23:31:28 +08:00

Release: 同步 develop 到 main（PR #1654~#1698 全量同步） #1699

Sign in to join this conversation.

Reviewers

No Reviewers

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

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#1691