opensource/kilocode
3
0
Fork 0
mirror of https://github.com/Kilo-Org/kilocode.git synced 2026-08-31 01:37:28 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(ui): defer popover dismiss listeners to prevent premature close in modals (#7367)

Browse Source
This commit is contained in:
Marius
2026-03-20 15:17:35 +01:00
committed by GitHub
parent 8769079a3f
commit da41fe6958
1 changed files with 12 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/ui/src/components/popover.tsx
+12 -2
View File
@@ -104,13 +104,23 @@ export function Popover<T extends ValidComponent = "div">(props: PopoverProps<T>
}
window.addEventListener("keydown", onKeyDown, true)
window.addEventListener("pointerdown", onPointerDown, true)
window.addEventListener("focusin", onFocusIn, true)
// Defer pointer/focus listeners so the portal content has time to mount
// and contentRef is assigned. Without this, autofocus or modal focus-trap
// events fire before the ref exists, causing the popover to immediately close.
const pending = {
id: requestAnimationFrame(() => {
pending.id = 0
window.addEventListener("pointerdown", onPointerDown, true)
window.addEventListener("focusin", onFocusIn, true)
}),
}
onCleanup(() => {
window.removeEventListener("keydown", onKeyDown, true)
window.removeEventListener("pointerdown", onPointerDown, true)
window.removeEventListener("focusin", onFocusIn, true)
if (pending.id) cancelAnimationFrame(pending.id)
})
})