sinvo/docmost
1
0
Fork 0
mirror of https://github.com/docmost/docmost.git synced 2026-05-18 23:44:24 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(base): ignore nested listbox and portal clicks so select doesnt close toolbar popover

Browse Source
This commit is contained in:
Philipinho
2026-04-18 23:31:53 +01:00
parent d9e2d7ba3d
commit f9bbbc7ebf
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
apps/client/src/features/base/components/base-toolbar.tsx
+8
View File
@@ -67,7 +67,15 @@ export function BaseToolbar({
const target = e.target as HTMLElement | null; const target = e.target as HTMLElement | null;
if (!target) return; if (!target) return;
if (toolbarRightRef.current?.contains(target)) return; if (toolbarRightRef.current?.contains(target)) return;
// Ignore clicks that land inside any Mantine popover dropdown
// (role=dialog), any Select/Combobox dropdown (role=listbox, the
// container; option elements have role=option), or anything
// rendered into Mantine's shared portal node. Without these, a
// nested Select inside the popover would close the parent.
if (target.closest('[role="dialog"]')) return; if (target.closest('[role="dialog"]')) return;
if (target.closest('[role="listbox"]')) return;
if (target.closest('[role="option"]')) return;
if (target.closest("[data-mantine-shared-portal-node]")) return;
closeAll(); closeAll();
}; };
const keyHandler = (e: KeyboardEvent) => { const keyHandler = (e: KeyboardEvent) => {