vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-05-06 09:10:45 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(clawsweeper): address review for automerge-openclaw-openclaw-75590 (1)

Browse Source
This commit is contained in:
clawsweeper
2026-05-03 02:10:43 +00:00
parent 175420b47f
commit 62240d8153
2 changed files with 18 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
ui/src/ui/chat/grouped-render.test.ts
View File

@@ -303,6 +303,14 @@ function openDeleteConfirm(deleteButton: HTMLButtonElement) {
deleteButton.dispatchEvent(new MouseEvent("click", { bubbles: true }));
}
function clickDeleteButtonIconPath(deleteButton: HTMLButtonElement) {
const icon = document.createElementNS("http://www.w3.org/2000/svg", "svg");
const path = document.createElementNS("http://www.w3.org/2000/svg", "path");
icon.appendChild(path);
deleteButton.appendChild(icon);
path.dispatchEvent(new MouseEvent("click", { bubbles: true }));
}
function setupArmedDeleteConfirm() {
const flushAnimationFrames = stubAnimationFrameQueue();
const addListenerSpy = vi.spyOn(document, "addEventListener");
@@ -455,6 +463,15 @@ describe("grouped chat rendering", () => {
expect(fixture.onDelete).not.toHaveBeenCalled();
});
it("removes the delete confirm outside-click listener when the delete button icon toggles it", () => {
const fixture = setupArmedDeleteConfirm();
clickDeleteButtonIconPath(fixture.deleteButton);
expectDeleteConfirmDismissed(fixture);
expect(fixture.onDelete).not.toHaveBeenCalled();
});
it("does not attach the delete confirm outside-click listener after an immediate toggle", () => {
const flushAnimationFrames = stubAnimationFrameQueue();
const addListenerSpy = vi.spyOn(document, "addEventListener");

2
ui/src/ui/chat/grouped-render.ts
View File

@@ -676,7 +676,7 @@ function renderDeleteButton(onDelete: () => void, side: DeleteConfirmSide) {
}
function closeOnOutside(evt: MouseEvent) {
const target = evt.target;
if (target instanceof Node && !popover.contains(target) && target !== btn) {
if (target instanceof Node && !popover.contains(target) && !btn.contains(target)) {
dismissPopover();
}
}