@atlaskit/top-layer 4.4.2 → 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;

package/dist/esm/focus/get-closest-layer.js

@@ -0,0 +1,4 @@
+export function getClosestLayer(_ref) {
+  var element = _ref.element;
+  return element.closest('[popover], dialog');
+}

package/dist/esm/focus/get-focusables.js

@@ -1,7 +1,32 @@
-import { isInSameLayer } from './is-in-same-layer';
-import { selectors } from './selectors';
+import { getClosestLayer } from './get-closest-layer';
+var focusableExclusions = ['[tabindex="-1"]', ':disabled', '[aria-disabled="true"]', '[aria-hidden="true"]'];
+var exclusionSelector = focusableExclusions.map(function (selector) {
+  return ":not(".concat(selector, ")");
+}).join('');
+var focusableSelector = ['a[href]', 'area[href]', 'input:not([type="hidden"])', 'select', 'textarea', 'button', 'iframe', 'audio[controls]', 'video[controls]', '[contenteditable]:not([contenteditable="false"])', '[tabindex]'].map(function (selector) {
+  return "".concat(selector).concat(exclusionSelector);
+}).join(',');
+function isVisible(element) {
+  // Visibility filtering must not break focus navigation when the API is unsupported.
+  if (typeof element.checkVisibility !== 'function') {
+    return true;
+  }
+  try {
+    // Exclude elements hidden by display, content-visibility or visibility because
+    // they cannot receive focus. Offscreen, occluded and opacity-zero elements can
+    // receive focus, so they intentionally remain included.
+    return element.checkVisibility({
+      checkVisibilityCSS: true
+    });
+  } catch (_unused) {
+    return true;
+  }
+}
+
 /**
  * Returns all focusable HTMLElements within the container, optionally filtered.
+ * Elements with `tabindex="-1"` are excluded because they can be focused
+ * programmatically but are not sequential focus navigation destinations.
  *
  * Focusables that belong to a nested top-layer scope (a `[popover]` or
  * `<dialog>` descendant of the container) are excluded - those elements
@@ -11,14 +36,24 @@ import { selectors } from './selectors';
 export function getFocusables(_ref) {
   var container = _ref.container,
     filter = _ref.filter;
-  return Array.from(container.querySelectorAll(selectors.focusable)).filter(function (element) {
-    return element instanceof HTMLElement;
-  }).filter(function (element) {
-    return isInSameLayer({
-      element: element,
-      container: container
-    });
-  }).filter(function (element) {
+  var containerLayer = getClosestLayer({
+    element: container
+  });
+  return Array.from(container.querySelectorAll(focusableSelector)).filter(function (element) {
+    if (!(element instanceof HTMLElement)) {
+      return false;
+    }
+    if (getClosestLayer({
+      element: element
+    }) !== containerLayer) {
+      return false;
+    }
+    if (element.closest('[inert]')) {
+      return false;
+    }
+    if (!isVisible(element)) {
+      return false;
+    }
     return !filter || filter(element, container);
   });
 }

package/dist/esm/focus/get-next-focusable.js

@@ -2,14 +2,17 @@
  * Returns the next focusable element in the given direction relative to the
  * currently focused element within the container. Wraps around at both ends.
  *
- * Returns `null` if there is no focused element within the container or if
- * the focused element is not in the list of focusable elements.
+ * The currently focused element does not need to be tabbable. This prevents
+ * navigation from getting stuck on an element that was programmatically or
+ * pointer focused with `tabindex="-1"`. Negative-tab-index elements remain
+ * excluded as destinations.
+ *
+ * Returns `null` if there is no focused element within the container.
  */
 
 import { getFocusables } from './get-focusables';
-import { selectors } from './selectors';
 export function getNextFocusable(_ref) {
-  var _all$nextIndex;
+  var _all3;
   var container = _ref.container,
     direction = _ref.direction,
     filter = _ref.filter;
@@ -17,21 +20,43 @@ export function getNextFocusable(_ref) {
     container: container,
     filter: filter
   });
-  var current = container.querySelector(selectors.focused);
-  if (!(current instanceof HTMLElement)) {
+  var first = all[0];
+  var last = all[all.length - 1];
+
+  // Checking the elements directly narrows types from `HTMLElement | undefined`.
+  if (!first || !last) {
     return null;
   }
-  var index = all.indexOf(current);
-  if (index === -1) {
+  var current = container.ownerDocument.activeElement;
+  if (!(current instanceof HTMLElement) || !container.contains(current)) {
     return null;
   }
-  var nextIndex = function () {
+  var index = all.indexOf(current);
+
+  // The focused element can be absent from `all` when it is non-tabbable or
+  // excluded by the filter. Find the first focusable element that follows it in
+  // the DOM, which identifies the nearest focusable destination in either direction.
+  if (index === -1) {
+    var _all;
+    var followingIndex = all.findIndex(function (element) {
+      return Boolean(current.compareDocumentPosition(element) & current.DOCUMENT_POSITION_FOLLOWING);
+    });
     if (direction === 'forwards') {
-      return (index + 1) % all.length;
+      var _all$followingIndex;
+      // If nothing follows the current element, then wrap to the start.
+      return (_all$followingIndex = all[followingIndex]) !== null && _all$followingIndex !== void 0 ? _all$followingIndex : first;
     }
-    var proposed = index - 1;
-    // if going into negative numbers, loop back to the last item
-    return proposed < 0 ? all.length - 1 : proposed;
-  }();
-  return (_all$nextIndex = all[nextIndex]) !== null && _all$nextIndex !== void 0 ? _all$nextIndex : null;
+
+    // The closest preceding element is immediately before the first following one.
+    // A missing adjacent entry means the backward sequence wraps to the end.
+    return (_all = all[followingIndex - 1]) !== null && _all !== void 0 ? _all : last;
+  }
+
+  // The current element is tabbable, so use its adjacent list entry and wrap
+  // when it is already at the boundary.
+  if (direction === 'forwards') {
+    var _all2;
+    return (_all2 = all[index + 1]) !== null && _all2 !== void 0 ? _all2 : first;
+  }
+  return (_all3 = all[index - 1]) !== null && _all3 !== void 0 ? _all3 : last;
 }

package/dist/esm/focus/is-nested-layer-focused.js

@@ -1,15 +1,13 @@
-import { getDocument } from '@atlaskit/browser-apis';
-import { isInSameLayer } from './is-in-same-layer';
+import { getClosestLayer } from './get-closest-layer';
 
 /**
  * Returns `true` when `document.activeElement` is inside a nested top-layer
- * descendant of `container` (a `[popover]`, `<dialog>`, `[role="dialog"]` or
- * `[role="alertdialog"]` that is itself a descendant of `container`).
+ * descendant of `container` (a `[popover]` or `<dialog>` that is itself a
+ * descendant of `container`).
  */
 export function isNestedLayerFocused(_ref) {
-  var _getDocument;
   var container = _ref.container;
-  var focused = (_getDocument = getDocument()) === null || _getDocument === void 0 ? void 0 : _getDocument.activeElement;
+  var focused = container.ownerDocument.activeElement;
   if (!(focused instanceof HTMLElement)) {
     return false;
   }
@@ -20,8 +18,9 @@ export function isNestedLayerFocused(_ref) {
   }
 
   // If not in the same layer, must be in a nested layer
-  return !isInSameLayer({
-    element: focused,
-    container: container
+  return getClosestLayer({
+    element: focused
+  }) !== getClosestLayer({
+    element: container
   });
 }