Files

5.0 KiB
Raw Permalink Blame History

id, title, phase, deps, status, created
id title phase deps status created
T-569 ②封面画廊上下条切换与未保存选择拦截 7
T-567
DONE 2026-07-09

问题 / 背景

T-567 的封面画廊只处理当前单条任务。运营批量检查时需要在弹窗内连续查看上一条/下一条商品,而不是关闭弹窗后回到列表再双击。切换记录时,如果当前画廊已经勾选了另一张候选但尚未保存,必须避免静默丢失选择。

方案(改哪个文件、改成什么)

上一条/下一条按钮

  • 在 T-567 封面画廊弹窗底部增加「上一条」「下一条」。
  • 目标任务来自②当前表格模型的相邻可见行,必须尊重当前筛选结果和排序。
  • 第一条时「上一条」置灰;最后一条时「下一条」置灰。
  • 切换记录时只刷新弹窗内容区和内部当前 task,不重建整个窗口;窗口大小和位置保持不变。

键盘切换

  • 支持 ↑ / ↓ 键切换上一条/下一条,和按钮走同一个切换函数。
  • 画廊内 QRadioButton 设置 ClickFocus 或 NoFocus,避免方向键在 radio 组内改变候选选择而不是切换记录。
  • 输入框/按钮等获得焦点时如不适合消费方向键,应保持 Qt 默认行为;不为键盘切换破坏可访问性。

未保存选择拦截

  • 判断条件:当前 radio 选中路径 != DB 当前 task.new_cover_path。
  • 在按钮切换、方向键切换、关闭弹窗前都必须进入同一套拦截流程。
  • 弹三态确认框,按钮必须是中文:
    • 「保存」:调用 db.update_generated_cover();如任务已提交线上,先弹 T-567 同款“本地换封面不会回滚蝦皮,重复更新会再次提交线上”确认;保存成功后继续切换/关闭。
    • 「不保存」:丢弃当前勾选,继续切换/关闭。
    • 「取消」:留在当前任务,不切换、不关闭。
  • 保存失败时保留当前任务和弹窗,显示中文错误。

与 T-568 的关系

  • 如果 T-568 已完成且当前任务正在单任务重生:
    • 可允许切换到其它任务,但切回时必须刷新最新 DB/候选;
    • 当前任务运行中时画廊显示“生成中”状态,不显示旧 new_cover_path 为当前有效图误导用户。
  • 如果 T-568 尚未完成,本任务只需保证 T-567 基础画廊切换逻辑正确。

验收要点

  • 上一条/下一条按钮按当前②可见列表相邻行切换;边界按钮置灰。
  • ↑ / ↓ 与按钮共用切换逻辑;方向键不会改变 radio 选中。
  • 切换记录不改变弹窗尺寸和屏幕位置。
  • 当前有未保存选择时,按钮切换、键盘切换、关闭弹窗都会弹中文三态确认。
  • 「保存 / 不保存 / 取消」三种路径行为正确;保存失败不切换。
  • 已提交线上任务保存前仍弹二次警示。
  • 验证命令:
    • py -3.10 -m unittest tests.test_gui
    • python -m ruff check app tests main.py
    • py -3.10 -m compileall app main.py
    • py -3.10 -m unittest discover -s tests
    • git diff --check

边界(不改什么)

  • 不改 T-566 数据层契约。
  • 不改 T-568 单任务重新生成逻辑。
  • 不改 cmhub/AI 请求、DB schema、Excel、③更新、CDP/Shopee。

执行记录

  • 2026-07-09 完成 T-569。
  • 修改 app/gui/tabs/generate.py:
    • ②封面画廊新增「上一条」「下一条」按钮,目标任务来自打开画廊时②当前可见表格任务列表,按相邻行切换,首尾边界按钮自动置灰。
    • 切换任务时只刷新弹窗内部任务、旧封面和候选画廊内容,不重建整个窗口,窗口尺寸和位置保持不变;切换时会重新读取 DB 最新任务和候选文件。
    • 支持 ↑ / ↓ 键走同一套上一条/下一条切换逻辑;候选 QRadioButton 改为 ClickFocus,避免方向键在 radio 组内改候选。
    • 增加统一未保存选择拦截:按钮切换、方向键切换、取消/关闭弹窗前,如果当前选中候选不同于当前 DB 封面指针,会弹中文三态确认「保存 / 不保存 / 取消」。保存路径复用 db.update_generated_cover(),已提交线上任务仍先弹“本地换封面不会回滚蝦皮,重复更新会再次提交线上”确认;保存失败保留当前任务。
    • 与 T-568 兼容:单任务封面重生运行中切换回该任务时显示“封面重新生成中”,完成/失败后继续刷新 DB 和候选列表。
  • 修改 tests/test_gui.py:
    • 覆盖上一条/下一条按当前可见列表切换、边界按钮状态、切换不改变窗口尺寸/位置。
    • 覆盖 ↑/↓ 键切换任务且不改变候选 radio 选择。
    • 覆盖未保存选择在切换和关闭时的「保存 / 不保存 / 取消」三种路径,以及已提交任务保存前的二次确认。
  • 验证通过:
    • py -3.10 -m unittest tests.test_gui
    • python -m ruff check app tests main.py
    • py -3.10 -m compileall app main.py
    • py -3.10 -m unittest discover -s tests(302 tests;PySide6 字体目录警告不影响结果)
    • git diff --check