GenarrativeAI/Genarrative
11
0
Fork 0
Code Issues 14 Pull Requests 3 Actions Packages Projects 3 Releases Wiki Activity

UI编辑器一批细节优化 #278

Merged
kdletters merged 44 commits from fix/ui-editor-minor-adjusts into master 2026-09-08 11:56:36 +08:00
Conversation 8 Commits 44 Files Changed 26
+8 -1
1 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit d456b97eaf - Show all commits
apps/ai-game-creator-shell/src/view/ui-editor/index.tsx
+8 -1
View File
@@ -88,6 +88,13 @@ export default function UiEditorPage({
k88936 marked this conversation as resolved
kdletters commented 2026-09-04 16:21:14 +08:00
Review
Copy Link
Copy Source

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。
kdletters commented 2026-09-04 16:21:14 +08:00
Review
Copy Link
Copy Source

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。
}, [session.save.isDirty]);
useEffect(() => {
const isEditableTarget = (target: EventTarget | null) => {
k88936 marked this conversation as resolved
kdletters commented 2026-09-04 16:21:14 +08:00
Review
Copy Link
Copy Source

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。
const element = target instanceof HTMLElement ? target : null;
k88936 marked this conversation as resolved
kdletters commented 2026-09-04 16:21:14 +08:00
Review
Copy Link
Copy Source

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。
return Boolean(
k88936 marked this conversation as resolved
kdletters commented 2026-09-04 16:21:14 +08:00
Review
Copy Link
Copy Source

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。
element?.isContentEditable ||
k88936 marked this conversation as resolved
kdletters commented 2026-09-04 16:21:14 +08:00
Review
Copy Link
Copy Source

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。
element?.closest('input, textarea, select, [contenteditable="true"]'),
k88936 marked this conversation as resolved
kdletters commented 2026-09-04 16:21:14 +08:00
Review
Copy Link
Copy Source

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。
);
k88936 marked this conversation as resolved
kdletters commented 2026-09-04 16:21:14 +08:00
Review
Copy Link
Copy Source

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。
};
k88936 marked this conversation as resolved
kdletters commented 2026-09-04 16:21:14 +08:00
Review
Copy Link
Copy Source

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。
const isInteractiveTarget = (target: EventTarget | null) => {
const element = target instanceof Element ? target : null;
return Boolean(
@@ -120,7 +127,7 @@ export default function UiEditorPage({
k88936 marked this conversation as resolved
kdletters commented 2026-09-04 16:21:14 +08:00
Review
Copy Link
Copy Source

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。
kdletters commented 2026-09-04 16:21:14 +08:00
Review
Copy Link
Copy Source

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。
if (
event.repeat ||
event.defaultPrevented ||
isInteractiveTarget(event.target) ||
k88936 marked this conversation as resolved
kdletters commented 2026-09-04 16:21:14 +08:00
Review
Copy Link
Copy Source

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。
isEditableTarget(event.target) ||
k88936 marked this conversation as resolved
kdletters commented 2026-09-04 16:21:14 +08:00
Review
Copy Link
Copy Source

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。
(!event.ctrlKey && !event.metaKey)
) {
return;
k88936 marked this conversation as resolved
kdletters commented 2026-09-04 16:21:14 +08:00
Review
Copy Link
Copy Source

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。
kdletters commented 2026-09-04 16:21:14 +08:00
Review
Copy Link
Copy Source

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。

完成通知弹窗打开时，页面级 keydown 监听仍会处理 Ctrl/Cmd+Z/Y：onKeyDown 只排除了可编辑元素，没有排除当前 role=dialog/弹窗内焦点。因此用户在弹窗的关闭按钮上按 Ctrl+Z（或 Ctrl+Shift+Z/Ctrl+Y）会直接撤销/重做底层 UI 编辑，弹窗声称阻塞工作台交互但状态仍被修改。建议在快捷键处理前检测 event.target 是否位于 role=dialog（或维护 modal-open 状态）并直接 return；同时补充弹窗打开时快捷键不改变编辑状态的测试。