UI编辑器一批细节优化 #278
Reference in New Issue
Block a user
Delete Branch "fix/ui-editor-minor-adjusts"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
提示词:完善组件建议,增加容器背景与节点大小一致的实现
支持Delete删除节点 ctrl + -缩放
步骤结束弹窗提醒


UI 树快捷删除

zoom滑动条样式
before:


after:
[head
fe813802a5] 发现 1 个明确问题,详见行内评论。完成通知弹窗打开时,页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y:onKeyDown 只排除了可编辑元素,没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z(或 Ctrl+Shift+Z/Ctrl+Y)会直接撤销/重做底层 UI 编辑,弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog(或维护 modal-open 状态)并直接 return;同时补充弹窗打开时快捷键不改变编辑状态的测试。
WIP: UI编辑器其他细节优化to WIP: UI编辑器一批细节优化[head
98c5768ebf] 审查结论:未发现明确问题。已检查 basef9e1e5d2a7到当前 head 的代码变更。[head
98c5768ebf] 审查结论:未发现明确问题。已检查 basef9e1e5d2a7到当前 head 的代码变更。[head
29b0b944bd] 审查结论:未发现明确问题。已检查 base524e2f0f4f到当前 head 的代码变更;前端 typecheck 通过。审查结论:request changes。除当前 Project CI 失败和与 master 冲突外,还有一个可复现的键盘误删问题:apps/ai-game-creator-shell/src/view/ui-editor/uiEditorKeyboardShortcuts.ts:22-24 对 data-ui-editor-zoom-controls 返回 false,导致其内部 range input 不被视为交互控件;同文件:44-65 的 Delete/Backspace 分支又早于 isEditableTarget 判断。于是用户聚焦缩放滑块时按 Delete 或 Backspace,会删除当前选中 UI 节点。请让缩放控件仍被 Delete 分支排除,同时保留所需的缩放快捷键行为,并补 range input 键盘回归测试。
审查结论:通过当前代码审查。上一轮关于缩放滑杆焦点下 Delete/Backspace 误删节点的问题已在当前提交通过交互控件判断和回归测试覆盖;弹窗焦点也已排除快捷键处理。当前四项 CI 仍在运行,合并前请等待 CI 结果。
请求修改(当前 head 777f928830a49d270800ea5d214475337ab850ee)。
代码审查层面上一轮键盘误删问题已修复,但当前 Project CI 的 Repository checks 失败:cargo fmt --check 报告 docs/technical/【设计】UI编辑器工作流完成通知弹窗-2026-09-04.md 第 41 行存在 EOF 多余空行。请清理该空行并重新运行仓库检查;修复后我会重新确认。
@@ -0,0 +38,4 @@3. 弹窗可通过标准关闭方式退出;关闭后卡片状态仍可见。4. 每条成功文案保留原有数量信息并增加可用的检查数量,所有文案包含“请检查”。5. 重新运行后新的终态会再次弹出通知。Repository checks 的 cargo fmt --check 已在此文件第 41 行报告 new blank line at EOF。请删除文件末尾多余空行后重跑 CI。
New commits pushed, approval review dismissed automatically according to repository settings
审查结论:request changes。
[P2] 缩放百分比无法编辑
文件:apps/ai-game-creator-shell/src/view/ui-editor/components/preview/PreviewWorkspace.tsx:548
当前把 displayPercent 渲染为触发 fit 的 ,点击会重新适配画布,用户无法直接编辑百分比;但本 PR 新增交互规范 docs/【交互设计】预览画布缩放滑杆-2026-09-05.md 明确要求“百分比文本可点击编辑并在失焦时限制到有效范围”。请改为受控输入(含失焦范围约束及与画布同步),并补相应组件级回归测试;或同步修正文档与验收测试使契约一致。