@atlaskit/top-layer 4.4.3 → 4.4.4

package/dist/es2019/popover/popover.js

@@ -1,4 +1,4 @@
-/* popover.tsx generated by @compiled/babel-plugin v2.0.0 */
+/* popover.tsx generated by @compiled/babel-plugin v3.0.1 */
 import "./popover.compiled.css";
 import { ax, ix } from "@compiled/react/runtime";
 import React, { forwardRef, useCallback, useId, useLayoutEffect, useMemo, useRef } from 'react';
@@ -89,7 +89,9 @@ const POPUP_ROLES = new Set(['menu', 'listbox', 'dialog', 'alertdialog', 'tree',
  * ### 📜 Browser dismiss is non-cancellable
  *
  * Per the popover spec, `beforetoggle` is cancellable on open transitions
- * only, not on close.
+ * only, not on close. Consumers should update controlled intent by setting
+ * `isOpen` to `false` from `onClose`. If controlled intent remains open after
+ * native close, the Popover remains closed until the prop changes.
  *
  * - Spec: <https://html.spec.whatwg.org/multipage/popover.html#the-popover-attribute>
  * - <https://github.com/whatwg/html/issues/8973>
@@ -128,45 +130,25 @@ export const Popover = /*#__PURE__*/forwardRef(function Popover({
   const combinedRef = mergeRefs([ownRef, ref].filter(Boolean));
   // `useId()` colons are invalid in CSS selectors and popover target attributes.
   const popoverId = idProp !== null && idProp !== void 0 ? idProp : `popover-${autoId.replace(/:/g, '')}`;
-  const {
-    phase,
-    onToggle
-  } = useAnimatedVisibility({
-    isOpen,
-    shouldAnimate,
-    elementRef: ownRef,
-    onEnterFinish,
-    onExitFinish
-  });
-
-  // True while the host element is mounted (any phase except `closed`).
-  // Used as a dep so listener-rebind effects re-attach after a remount.
-  const isVisible = phase !== 'closed';
 
-  // Focus management: initial focus on entry (role-dependent), Tab cycling
-  // for dialog roles. Restoration is native for outermost popovers; nested
-  // focus-capturing roles are handled below via the `beforetoggle` snapshot.
-  // Passing `phase` keeps `useFocusWrap` attached through the animated-exit
-  // window (WCAG 2.4.3 guard). See notes/architecture/focus-restoration.md.
-  useFocusWrap({
-    elementRef: ownRef,
-    role,
-    phase
-  });
-  useInitialFocus({
-    elementRef: ownRef,
-    phase,
-    role
-  });
+  // Keep semantic callbacks current without rebinding native lifecycle
+  // listeners during the same commit that calls showPopover()/hidePopover().
+  const onCloseRef = useRef(onClose);
+  const roleRef = useRef(role);
+  useLayoutEffect(() => {
+    onCloseRef.current = onClose;
+    roleRef.current = role;
+  }, [onClose, role]);
 
   // Register with open layer observer so `closeLayers()` and open-count
   // subscriptions work. Only popup-like roles register as `popup`; passive
   // roles (tooltip, status, etc.) leave `type` undefined.
   const handleObserverClose = useCallback(() => {
-    onClose === null || onClose === void 0 ? void 0 : onClose({
+    var _onCloseRef$current;
+    (_onCloseRef$current = onCloseRef.current) === null || _onCloseRef$current === void 0 ? void 0 : _onCloseRef$current.call(onCloseRef, {
       reason: 'programmatic'
     });
-  }, [onClose]);
+  }, []);
   useNotifyOpenLayerObserver({
     type: role && POPUP_ROLES.has(role) ? 'popup' : undefined,
     isOpen,
@@ -208,51 +190,76 @@ export const Popover = /*#__PURE__*/forwardRef(function Popover({
     const active = document.activeElement;
     previouslyFocusedElementRef.current = active instanceof HTMLElement ? active : null;
   }, []);
-  const handleToggle = useCallback(event => {
-    const element = ownRef.current;
-    if (event.newState === 'open') {
-      onToggle(event);
-      return;
+  const restorePreviouslyFocusedElement = useCallback(() => {
+    const previouslyFocused = previouslyFocusedElementRef.current;
+    previouslyFocusedElementRef.current = null;
+    if (previouslyFocused !== null && previouslyFocused.isConnected && shouldFocusIntoPopover({
+      role: roleRef.current
+    })) {
+      previouslyFocused.focus({
+        preventScroll: true
+      });
+    }
+  }, []);
+  const handleToggleClosed = useCallback(_event => {
+    var _onCloseRef$current2;
+    // Nested-popover focus restoration fallback (browser only
+    // restores outermost; Firefox skips nested entirely).
+    // Restore only when the role moved focus in on open AND the
+    // close was Escape/programmatic. For light dismiss, the
+    // click target keeps focus per HTML spec
+    // (`focusPreviousElement=false`).
+    const reason = closeReasonRef.current;
+    const isProgrammatic = programmaticCloseRef.current;
+    if (reason === 'escape' || isProgrammatic) {
+      restorePreviouslyFocusedElement();
+    } else {
+      previouslyFocusedElementRef.current = null;
     }
-    if (event.newState === 'closed') {
-      if (element) {
-        // Nested-popover focus restoration fallback (browser only
-        // restores outermost; Firefox skips nested entirely).
-        // Restore only when the role moved focus in on open AND the
-        // close was Escape/programmatic. For light dismiss, the
-        // click target keeps focus per HTML spec
-        // (`focusPreviousElement=false`).
-        const previouslyFocused = previouslyFocusedElementRef.current;
-        const reason = closeReasonRef.current;
-        const isProgrammatic = programmaticCloseRef.current;
-        const shouldRestore = previouslyFocused !== null && previouslyFocused.isConnected && shouldFocusIntoPopover({
-          role
-        }) && (reason === 'escape' || isProgrammatic);
-        if (shouldRestore && previouslyFocused) {
-          previouslyFocused.focus({
-            preventScroll: true
-          });
-        }
-        previouslyFocusedElementRef.current = null;
-      }
 
-      // Programmatic close: consumer already knows.
-      if (programmaticCloseRef.current) {
-        // Reset reason so a stale 'escape' from a race with Escape keydown
-        // does not corrupt the next browser-dismiss cycle.
-        closeReasonRef.current = 'light-dismiss';
-      } else {
-        // Browser dismiss (Escape/click-outside).
-        const reason = closeReasonRef.current;
-        closeReasonRef.current = 'light-dismiss';
-        // Optional: `manual` mode has no `onClose` in the forwarded type.
-        onClose === null || onClose === void 0 ? void 0 : onClose({
-          reason
-        });
-      }
+    // Reset reason so a stale 'escape' from a race with Escape keydown
+    // does not corrupt the next browser-dismiss cycle.
+    closeReasonRef.current = 'light-dismiss';
+
+    // Programmatic close: consumer already knows.
+    if (isProgrammatic) {
+      return;
     }
-    onToggle(event);
-  }, [onClose, onToggle, role]);
+
+    // Browser dismiss (Escape/click-outside).
+    // Optional: `manual` mode has no `onClose` in the forwarded type.
+    (_onCloseRef$current2 = onCloseRef.current) === null || _onCloseRef$current2 === void 0 ? void 0 : _onCloseRef$current2.call(onCloseRef, {
+      reason
+    });
+  }, [restorePreviouslyFocusedElement]);
+  const {
+    phase,
+    isMounted,
+    onBeforeToggle,
+    onToggle
+  } = useAnimatedVisibility({
+    isOpen,
+    shouldAnimate,
+    elementRef: ownRef,
+    onEnterFinish,
+    onExitFinish
+  });
+
+  // Focus management: initial focus on entry (role-dependent), Tab cycling
+  // for dialog roles. Restoration is native for outermost popovers; nested
+  // focus-capturing roles are handled above via the `beforetoggle` snapshot.
+  // Passing `phase` keeps `useFocusWrap` attached through the animated-exit
+  // window (WCAG 2.4.3 guard). See notes/architecture/focus-restoration.md.
+  useFocusWrap({
+    elementRef: ownRef,
+    role,
+    phase
+  });
+  useInitialFocus({
+    elementRef: ownRef,
+    phase,
+    role
+  });
 
   // Bind via `useLayoutEffect`, and BEFORE the show/hide layout effect
   // below, so listeners are attached when `showPopover()` synchronously
@@ -280,19 +287,28 @@ export const Popover = /*#__PURE__*/forwardRef(function Popover({
     });
     const unbindToggle = bind(element, {
       type: 'toggle',
-      listener: handleToggle
+      listener: event => {
+        if (event.newState === 'closed') {
+          handleToggleClosed(event);
+        }
+        onToggle(event);
+      }
     });
-    // Snapshot `document.activeElement` before `useInitialFocus` moves it.
+    // Snapshot `document.activeElement` before `useInitialFocus` moves it,
+    // then advance the shared visibility lifecycle.
     const unbindBeforeToggle = bind(element, {
       type: 'beforetoggle',
-      listener: handleBeforeToggle
+      listener: event => {
+        handleBeforeToggle(event);
+        onBeforeToggle(event);
+      }
     });
     return () => {
       unbindEscape();
       unbindToggle();
       unbindBeforeToggle();
     };
-  }, [handleToggle, handleBeforeToggle, isVisible]);
+  }, [handleBeforeToggle, handleToggleClosed, isMounted, onBeforeToggle, onToggle]);
 
   // Show/hide based on isOpen. `showPopover`/`hidePopover` are no-ops when
   // already in the target state. Try/catch guards `InvalidStateError` on
@@ -305,8 +321,8 @@ export const Popover = /*#__PURE__*/forwardRef(function Popover({
     if (isOpen) {
       programmaticCloseRef.current = false;
       // Clear stale 'escape' from a prior cycle: the ref outlives
-      // the host element, and the no-animation path unmounts before
-      // `handleToggle` can reset it.
+      // the host element, and interrupted close lifecycles may not reach
+      // `handleToggleClosed` to reset it.
       closeReasonRef.current = 'light-dismiss';
       try {
         element.showPopover();
@@ -327,7 +343,7 @@ export const Popover = /*#__PURE__*/forwardRef(function Popover({
   // Unmount the host once exit completes so it does not leave an empty
   // `role`/`popover` element in the accessibility tree. The element
   // remounts on the next open; the `popoverId` is stable across opens.
-  if (!isVisible) {
+  if (!isMounted) {
     return null;
   }
   return /*#__PURE__*/React.createElement("div", {

package/dist/es2019/popover-surface/popover-surface.js

@@ -1,4 +1,4 @@
-/* popover-surface.tsx generated by @compiled/babel-plugin v2.0.0 */
+/* popover-surface.tsx generated by @compiled/babel-plugin v3.0.1 */
 import "./popover-surface.compiled.css";
 import * as React from 'react';
 import { ax, ix } from "@compiled/react/runtime";

package/dist/esm/dialog/dialog-content.js

@@ -1,4 +1,4 @@
-/* dialog-content.tsx generated by @compiled/babel-plugin v2.0.0 */
+/* dialog-content.tsx generated by @compiled/babel-plugin v3.0.1 */
 import "./dialog-content.compiled.css";
 import { ax, ix } from "@compiled/react/runtime";
 import React, { forwardRef, useCallback, useId, useLayoutEffect, useRef } from 'react';
@@ -76,6 +76,13 @@ export var Dialog = /*#__PURE__*/forwardRef(function Dialog(_ref, ref) {
   var ownRef = useRef(null);
   var combinedRef = mergeRefs([ownRef, ref]);
   var closeReasonRef = useRef(null);
+  var dismissedByRef = useRef(dismissedBy);
+  var onCloseRef = useRef(onClose);
+  // Keep native event handlers stable while ensuring they read the latest committed props.
+  useLayoutEffect(function () {
+    dismissedByRef.current = dismissedBy;
+    onCloseRef.current = onClose;
+  }, [dismissedBy, onClose]);
   var _useAnimatedVisibilit = useAnimatedVisibility({
       isOpen: isOpen,
       shouldAnimate: shouldAnimate,
@@ -84,11 +91,52 @@ export var Dialog = /*#__PURE__*/forwardRef(function Dialog(_ref, ref) {
       onExitFinish: onExitFinish
     }),
     phase = _useAnimatedVisibilit.phase,
+    isMounted = _useAnimatedVisibilit.isMounted,
+    onBeforeToggle = _useAnimatedVisibilit.onBeforeToggle,
     onToggle = _useAnimatedVisibilit.onToggle;
 
-  // True while the host element is mounted (any phase except `closed`).
-  // Used as a dep so listener-rebind effects re-attach after a remount.
-  var isVisible = phase !== 'closed';
+  // Ordering is important: bind native lifecycle listeners before the later
+  // layout effect calls showModal() or close(), because `beforetoggle` fires
+  // synchronously during those calls.
+  useLayoutEffect(function () {
+    if (!isMounted) {
+      return;
+    }
+    var dialog = ownRef.current;
+    if (!dialog) {
+      return;
+    }
+    return bindAll(dialog, [{
+      type: 'beforetoggle',
+      listener: onBeforeToggle
+    }, {
+      // Browsers retarget clicks on ::backdrop to the <dialog> element.
+      // Close on backdrop click when outside-click dismissal is enabled.
+      type: 'click',
+      listener: function listener(event) {
+        if (event.target === event.currentTarget && dismissedByRef.current === 'escape-and-outside-click') {
+          closeReasonRef.current = 'overlay-click';
+          dialog.close();
+        }
+      }
+    }, {
+      type: 'toggle',
+      listener: function listener(event) {
+        if (event.newState === 'closed') {
+          var reason = closeReasonRef.current;
+          closeReasonRef.current = null;
+          // A reason is only set for user-initiated closes. When React closes the
+          // dialog because isOpen changed, the consumer already knows.
+          if (reason) {
+            onCloseRef.current({
+              reason: reason
+            });
+          }
+        }
+        onToggle(event);
+      }
+    }]);
+  }, [isMounted, onBeforeToggle, onToggle]);
 
   // Native `<dialog>.showModal()` traps focus but wraps through `<body>` at
   // the boundary (A → B → C → body → A). This hook intercepts Tab to wrap
@@ -134,7 +182,7 @@ export var Dialog = /*#__PURE__*/forwardRef(function Dialog(_ref, ref) {
   // See notes/decisions/safari-escape-nested-popover-in-dialog.md
   var _useSafariEscapeFix = useSafariEscapeFix({
       dialogRef: ownRef,
-      isVisible: isVisible
+      isVisible: isMounted
     }),
     shouldIgnoreEscape = _useSafariEscapeFix.shouldIgnoreEscape;
 
@@ -167,49 +215,11 @@ export var Dialog = /*#__PURE__*/forwardRef(function Dialog(_ref, ref) {
     closeReasonRef.current = 'escape';
   }, [dismissedBy, shouldIgnoreEscape]);
 
-  // Handle backdrop interaction
-  // Attached via bind-event-listener rather than a React prop so we avoid
-  // a11y lint suppressions on the <dialog> element.
-  // The click listener provides light dismiss consistently across browsers.
-  // `isVisible` is in deps so the listener re-binds to the new <dialog>
-  // element after a host unmount / remount cycle.
-  useLayoutEffect(function () {
-    var dialog = ownRef.current;
-    if (!dialog) {
-      return;
-    }
-    return bindAll(dialog, [{
-      type: 'click',
-      listener: function listener(event) {
-        if (event.target === event.currentTarget && dismissedBy === 'escape-and-outside-click') {
-          closeReasonRef.current = 'overlay-click';
-          // Calls native close because this is emulating the native light-dismiss behavior
-          dialog.close();
-        }
-      }
-    }, {
-      type: 'toggle',
-      listener: function listener(event) {
-        if (event.newState === 'closed') {
-          var reason = closeReasonRef.current;
-          closeReasonRef.current = null;
-          if (reason) {
-            onClose({
-              reason: reason
-            });
-          }
-        }
-
-        // Visibility cleanup runs after close-reason handling.
-        onToggle(event);
-      }
-    }]);
-  }, [dismissedBy, onClose, onToggle, isVisible]);
-
   // Unmount the `<dialog>` once exit completes so it does not leave an
-  // empty `role="dialog"` element in the accessibility tree. On the next
-  // open the element remounts and its effects run against the fresh element.
-  if (!isVisible) {
+  // empty `role="dialog"` element in the accessibility tree. A later open
+  // remounts a fresh element, so the listener-binding and native-visibility
+  // effects run against that element.
+  if (!isMounted) {
     return null;
   }
   var shouldAnimateBackdrop = !shouldHideBackdrop && shouldAnimate;