Skip to content

全面重构 UI 设计系统 + 移除指纹匹配 + 修复节点列表布局#98

Merged
AfishInLake merged 1 commit into
masterfrom
feature/render
Jun 25, 2026
Merged

全面重构 UI 设计系统 + 移除指纹匹配 + 修复节点列表布局#98
AfishInLake merged 1 commit into
masterfrom
feature/render

Conversation

@AfishInLake

Copy link
Copy Markdown
Owner
  • UI 设计 token 统一:硬编码色值替换为暖色调 CSS 自定义属性(brass/accent 主题)
  • 配置 UI 补全:新增 15 个此前缺失的 Pydantic 模型字段
  • DocTagChecker 布局:固定 100vh,仅节点列表滚动,详情面板始终可见
  • 节点项简化:flex 单行 + 固定宽度列约束,长文本内部换行撑高
  • 过滤无内容 figure_image 节点
  • 后端:移除 XML 指纹匹配,改为位置顺序绑定(zip)
  • 样式定义修正:段落属性(段前/段后距、行距、缩进)写入 XML 样式定义

📌 关联 Issue

  • Closes #xxx
  • Related to #yyy

🎯 变更概述

  • 兼容现有功能(默认选项)
  • 包含破坏性更新(如勾选,请完整填写以下内容)

⚠️ 破坏性更新说明(仅当上方勾选时填写)

  • 受影响的用户场景

    (例如:使用 v0.2 格式配置文件的用户升级后将遇到解析错误)

  • 具体变更点

    (例如:YAML 配置中 style.font_size 字段已移除,改为嵌套结构 style.font.size

  • 迁移建议

    (例如:请将 font_size: 12 更新为 font: { size: 12 }

  • 是否提供自动迁移工具?

    • 是(路径:scripts/migrate_v02_to_v03.py

✅ 提交前自查清单

🔹 测试保障

  • 已通过 make test(或等效命令)
  • 新增/修改的功能有对应单元测试(位于 tests/ 目录)
  • 所有测试均使用 io.BytesIO,未保存任何临时 .docx 文件到磁盘
  • 测试覆盖了正常路径、边界情况和错误输入

🔹 配置与兼容性

  • YAML 配置变更向后兼容(旧配置仍能运行,或提供迁移说明)
  • 新增配置项已在 example/ 中提供示例
  • 不依赖未声明的第三方库(requirements.txtpyproject.toml 已更新)

🔹 文档与示例

  • README.md 已更新(如新增功能说明、命令参数)
  • 示例文档(example/)能正常运行且反映最新能力
  • 复杂逻辑已添加必要注释(尤其是涉及 python-docx 底层操作的部分)

🔹 安全与资源

  • 无敏感信息泄露(如 API Key、本地路径)
  • 无内存泄漏风险(如大文档处理时及时释放对象)
  • 未使用不安全的反序列化(如 pickleyaml.load 未指定 Loader

🔹 其他

  • 分支基于最新 master 切出,无无关提交
  • Commit 信息清晰(建议遵循 Conventional Commits,如 feat: add table caption check

💡 温馨提示:勾选 [x] 表示你已确认该项。未勾选项可能成为 Review 阻塞点。

- UI 设计 token 统一:硬编码色值替换为暖色调 CSS 自定义属性(brass/accent 主题)
- 配置 UI 补全:新增 15 个此前缺失的 Pydantic 模型字段
- DocTagChecker 布局:固定 100vh,仅节点列表滚动,详情面板始终可见
- 节点项简化:flex 单行 + 固定宽度列约束,长文本内部换行撑高
- 过滤无内容 figure_image 节点
- 后端:移除 XML 指纹匹配,改为位置顺序绑定(zip)
- 样式定义修正:段落属性(段前/段后距、行距、缩进)写入 XML 样式定义
@AfishInLake AfishInLake merged commit ce0021f into master Jun 25, 2026
12 checks passed
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.

1 participant