src-advisory/SRCmail
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: context menu invisible on first right-click after page load

Browse Source
This commit is contained in:
Linus Rath
2026-06-12 00:21:09 +02:00
parent fe5645c818
commit e1c28e767a
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
components/ui/context-menu.tsx
+4 -1
View File
@@ -32,6 +32,9 @@ export const ContextMenu = forwardRef<HTMLDivElement, ContextMenuProps>(
// Measure the rendered menu and clamp it inside the viewport before the // Measure the rendered menu and clamp it inside the viewport before the
// browser paints. We hide the element until this runs so the user never // browser paints. We hide the element until this runs so the user never
// sees the menu jump from an unclamped position to a clamped one. // sees the menu jump from an unclamped position to a clamped one.
// `mounted` must be a dependency: on the very first open the menu isn't
// in the DOM yet (mounted is still false), so this effect has to re-run
// after the mount flip or the menu stays visibility:hidden.
useLayoutEffect(() => { useLayoutEffect(() => {
if (!isOpen) { if (!isOpen) {
setAdjustedPosition(null); setAdjustedPosition(null);
@@ -57,7 +60,7 @@ export const ContextMenu = forwardRef<HTMLDivElement, ContextMenuProps>(
y = Math.max(VIEWPORT_MARGIN, y); y = Math.max(VIEWPORT_MARGIN, y);
setAdjustedPosition({ x, y }); setAdjustedPosition({ x, y });
}, [isOpen, position.x, position.y]); }, [isOpen, mounted, position.x, position.y]);
const setRefs = (node: HTMLDivElement | null) => { const setRefs = (node: HTMLDivElement | null) => {
localRef.current = node; localRef.current = node;