@atlaskit/top-layer 0.12.1 → 0.12.2

package/dist/esm/dialog/create-close-event.js

@@ -2,7 +2,7 @@
  * Creates a synthetic DOM event from a `TDialogCloseReason`.
  *
  * Useful when bridging the Dialog primitive's `onClose({ reason })` callback
- * to a legacy API that expects a `KeyboardEvent` or `MouseEvent` — for example,
+ * to a legacy API that expects a `KeyboardEvent` or `MouseEvent` - for example,
  * `@atlaskit/modal-dialog`'s `onClose(event, analyticsEvent)` contract.
  *
  * - `'escape'` → `KeyboardEvent('keydown', { key: 'Escape' })`

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

@@ -49,7 +49,7 @@ export var Dialog = /*#__PURE__*/forwardRef(function Dialog(_ref, ref) {
   var ownRef = useRef(null);
   var combinedRef = mergeRefs([ownRef, ref]);
 
-  // ── Animation lifecycle ──
+  // Animation lifecycle
   var _useAnimatedVisibilit = useAnimatedVisibility({
       isOpen: isOpen,
       animate: animate,
@@ -59,7 +59,7 @@ export var Dialog = /*#__PURE__*/forwardRef(function Dialog(_ref, ref) {
     showChildren = _useAnimatedVisibilit.showChildren,
     preset = _useAnimatedVisibilit.preset;
 
-  // ── Focus wrap ──
+  // Focus wrap
   // Native <dialog>.showModal() traps focus but wraps through <body> at
   // the boundary (A → B → C → body → A). This hook intercepts Tab to
   // wrap directly (A → B → C → A), matching the WAI-ARIA APG pattern.
@@ -68,7 +68,7 @@ export var Dialog = /*#__PURE__*/forwardRef(function Dialog(_ref, ref) {
     role: 'dialog'
   });
 
-  // ── Open layer observer registration ──
+  // Open layer observer registration
   // Notifies the open layer observer so app-coordination features
   // (open-count subscriptions) work with top-layer dialogs.
   useNotifyOpenLayerObserver({
@@ -99,7 +99,7 @@ export var Dialog = /*#__PURE__*/forwardRef(function Dialog(_ref, ref) {
     }
   }, [isOpen]);
 
-  // ── Handle native Escape (cancel event) ──
+  // Handle native Escape (cancel event)
   var handleCancel = useCallback(function (event) {
     event.preventDefault();
     onClose({
@@ -107,7 +107,7 @@ export var Dialog = /*#__PURE__*/forwardRef(function Dialog(_ref, ref) {
     });
   }, [onClose]);
 
-  // ── Handle backdrop click ──
+  // Handle backdrop click
   // Attached via bind-event-listener rather than a React prop so we avoid
   // a11y lint suppressions on the <dialog> element.
   // Keyboard dismiss is already handled natively (Escape → onCancel above).

package/dist/esm/internal/ensure-preset-styles.js

@@ -4,7 +4,7 @@
  * Each preset's CSS is injected at most once into `<head>`.
  * Shared by both PopoverContent and DialogContent.
  *
- * Note: injected `<style>` elements and the tracking Set are append-only —
+ * Note: injected `<style>` elements and the tracking Set are append-only -
  * they are never removed. This is intentional: the number of distinct
  * presets is bounded (handful of animation presets), so the
  * memory footprint is negligible. Removing styles would risk breaking

package/dist/esm/internal/reduced-motion.js

@@ -7,7 +7,7 @@ import once from '@atlaskit/ds-lib/once';
  *
  * The result is cached after the first call via `once`. This avoids
  * calling `matchMedia` on every render. The trade-off is that if a user
- * toggles the OS reduced-motion setting mid-session, the change won't
+ * toggles the OS reduced-motion setting mid-session, the change will not
  * take effect until the next page load. This is acceptable because
  * toggling this system-level preference mid-session is extremely rare,
  * and a page refresh is a reasonable expectation for system accessibility

package/dist/esm/internal/resolve-placement.js

@@ -25,8 +25,8 @@ var DEFAULT_GAP = "var(--ds-space-100, 8px)";
  * CSS length string (e.g. `token('space.100')`).
  *
  * The two offset axes are named for clarity at the call site:
- *   - `gap`     — distance away from the trigger along the placement axis
- *   - `crossAxisShift`  — nudge along the trigger edge (perpendicular to the placement axis)
+ *   - `gap`     - distance away from the trigger along the placement axis
+ *   - `crossAxisShift`  - nudge along the trigger edge (perpendicular to the placement axis)
  */
 
 /**

package/dist/esm/internal/role-types.js

@@ -22,7 +22,7 @@
  * (e.g. a combobox labels its listbox, tooltip content IS the accessible name).
  */
 
-// ── Shared ARIA role discrimination types ──
+// Shared ARIA role discrimination types
 //
 // These encode the WCAG 4.1.2 constraint: roles like `dialog`, `alertdialog`,
 // and `menu` MUST have an accessible name (either `label` or `labelledBy`).

package/dist/esm/internal/slot.js

@@ -26,14 +26,14 @@ function isSlotChild(child) {
  * Merges props (including ref) onto a single React element child.
  *
  * A safer alternative to raw `cloneElement` that:
- * 1. Guards with `isValidElement` — non-element children (strings,
+ * 1. Guards with `isValidElement` - non-element children (strings,
  *    fragments, null) are returned as-is instead of crashing.
  * 2. Merges refs via `mergeRefs` so both the caller's ref and the
  *    child's existing ref are honoured.
  * 3. Spreads any additional props (ARIA attributes, event handlers, etc.)
  *    onto the child element.
  *
- * Event handlers (e.g. `onClick`) are **not** auto-merged — the caller's
+ * Event handlers (e.g. `onClick`) are **not** auto-merged - the caller's
  * handler replaces the child's. Compose handlers before passing them in.
  *
  * @example
@@ -46,7 +46,7 @@ function isSlotChild(child) {
 export var Slot = /*#__PURE__*/forwardRef(function Slot(slotProps, ref) {
   var rawChildren = slotProps.children,
     props = _objectWithoutProperties(slotProps, _excluded);
-  // `SlotProps` intersects `Record<string, unknown>`, so `children` is typed as `unknown`.
+  // `TSlotProps` intersects `Record<string, unknown>`, so `children` is typed as `unknown`.
   var children = rawChildren;
   if (!isSlotChild(children)) {
     return children;

package/dist/esm/internal/use-anchor-position.js

@@ -1,5 +1,3 @@
-import _classCallCheck from "@babel/runtime/helpers/classCallCheck";
-import _createClass from "@babel/runtime/helpers/createClass";
 /**
  * Anchor positioning for top-layer popovers. See
  * `notes/architecture/positioning.md` for the full model.
@@ -30,25 +28,15 @@ import { setStyle } from './set-style';
 // from `./resolve-placement` so search-and-jump lands at the source of truth.
 
 /**
- * Module-scope no-op `ResizeObserver` used in non-DOM jest environments where
+ * Module-scope no-op `ResizeObserver` stub used in non-DOM jest environments where
  * `ResizeObserver` is `undefined`. Hoisted out of the effect to avoid
- * allocating a new class declaration on every run.
+ * re-allocating on every run.
  */
-var NoopResizeObserver = /*#__PURE__*/function () {
-  function NoopResizeObserver() {
-    _classCallCheck(this, NoopResizeObserver);
-  }
-  return _createClass(NoopResizeObserver, [{
-    key: "observe",
-    value: function observe() {}
-  }, {
-    key: "unobserve",
-    value: function unobserve() {}
-  }, {
-    key: "disconnect",
-    value: function disconnect() {}
-  }]);
-}();
+var noopResizeObserver = {
+  observe: function observe() {},
+  unobserve: function unobserve() {},
+  disconnect: function disconnect() {}
+};
 
 /**
  * Returns true when the popover is currently open. Wrapped because the
@@ -338,7 +326,7 @@ export function useAnchorPosition(_ref3) {
       return _undoPositioning;
     }
 
-    // ── JS fallback ──
+    // JS fallback
     // The popover is already in the browser's top layer via popover="auto",
     // so it does not need position:fixed. We only reset the UA defaults
     // (inset: 0; margin: auto) and set top/left based on measurements.
@@ -407,20 +395,17 @@ export function useAnchorPosition(_ref3) {
     // Self-disconnects after one valid measurement; ongoing
     // scroll/resize is handled by the window listeners below.
     // `ResizeObserver` is missing in some non-DOM jest environments
-    // (e.g. post-office's `node` environment). Fall back to a no-op
-    // observer there. The scroll/resize listeners below still keep
-    // the popover positioned in the rare case the consumer also
-    // polyfilled `showPopover` but not `ResizeObserver`. Real
-    // browsers always have it. The `NoopResizeObserver` class is
-    // hoisted to module scope (see top of file) so we do not allocate
-    // a new constructor on every effect run.
-    var ResizeObserverImpl = typeof ResizeObserver !== 'undefined' ? ResizeObserver : NoopResizeObserver;
-    var resizeObserver = new ResizeObserverImpl(function () {
+    // (e.g. post-office's `node` environment). Fall back to the
+    // module-scope `noopResizeObserver` stub there. The scroll/resize
+    // listeners below still keep the popover positioned in the rare
+    // case the consumer also polyfilled `showPopover` but not
+    // `ResizeObserver`. Real browsers always have it.
+    var resizeObserver = typeof ResizeObserver !== 'undefined' ? new ResizeObserver(function () {
       if (popover.offsetWidth > 0 && popover.offsetHeight > 0) {
         update();
         resizeObserver.disconnect();
       }
-    });
+    }) : noopResizeObserver;
 
     // If the popover is already open by the time this effect runs (for
     // example, the parent `useAnchorPosition` effect runs after the child

package/dist/esm/internal/use-animated-visibility.js

@@ -8,19 +8,19 @@ import { usePresetStyles } from './use-preset-styles';
  *
  * Used by both `Popover` and `Dialog` to share the same animation lifecycle
  * logic. Those components own their own show/hide mechanisms (showPopover /
- * hidePopover vs showModal / close) and event handling — this hook only
+ * hidePopover vs showModal / close) and event handling - this hook only
  * manages the relationship between `isOpen` and when children are rendered.
  *
  * ## Problem
  *
  * We want to delay the unmount of children until a CSS exit transition has
  * finished playing. If we unmount children the moment `isOpen` becomes
- * `false`, the exit animation is never visible — the content just disappears.
+ * `false`, the exit animation is never visible - the content just disappears.
  *
  * ## How it works
  *
  * `showChildren` tracks whether the consumer's children should be in the DOM.
- * It does not mirror `isOpen` exactly — it intentionally lags behind during
+ * It does not mirror `isOpen` exactly - it intentionally lags behind during
  * exit animations:
  *
  * ```
@@ -56,12 +56,12 @@ export function useAnimatedVisibility(_ref) {
   // or the user has enabled the "prefers reduced motion" OS setting.
   var willAnimate = Boolean(animate) && !prefersReducedMotion();
 
-  // ── showChildren state ──
+  // showChildren state
   //
   // This is the core of the hook. `showChildren` controls whether the
   // consumer's children are in the DOM. The two `if` blocks below are
   // intentionally written as synchronous state updates during render
-  // (React supports this pattern — setState during render is treated as
+  // (React supports this pattern - setState during render is treated as
   // a synchronous re-render before the browser paints).
   //
   // Mount path:  isOpen went true but showChildren is still false → mount.
@@ -88,7 +88,7 @@ export function useAnimatedVisibility(_ref) {
   var onExitFinishRef = useRef(onExitFinish);
   onExitFinishRef.current = onExitFinish;
 
-  // ── Animated entry effect ──
+  // Animated entry effect
   //
   // Fires `onEnterFinish` after the entry transition completes.
   // When there is no animation, fires it immediately via a separate effect.
@@ -137,7 +137,7 @@ export function useAnimatedVisibility(_ref) {
     };
   }, [isOpen, showChildren, willAnimate, preset === null || preset === void 0 ? void 0 : preset.enterDurationMs, elementRef]);
 
-  // ── Non-animated entry callback ──
+  // Non-animated entry callback
   //
   // When there is no animation, fire `onEnterFinish` immediately on mount and after `isOpen` goes false->true.
   // The initial value is false to make sure it fires on mount if the current `isOpen` is true.
@@ -151,7 +151,7 @@ export function useAnimatedVisibility(_ref) {
     }
   }, [isOpen, willAnimate]);
 
-  // ── Animated exit effect ──
+  // Animated exit effect
   //
   // Runs when all three conditions are true:
   //   • `isOpen` is false  (consumer wants to close)
@@ -214,10 +214,10 @@ export function useAnimatedVisibility(_ref) {
     };
   }, [isOpen, showChildren, willAnimate, preset === null || preset === void 0 ? void 0 : preset.exitDurationMs, elementRef]);
 
-  // ── Non-animated exit callback ──
+  // Non-animated exit callback
   //
   // When there is no animation, children unmount synchronously (handled
-  // by the `if` block above). But `onExitFinish` still needs to fire —
+  // by the `if` block above). But `onExitFinish` still needs to fire -
   // we do that in a follow-up effect so the consumer can react to it.
   //
   // Uses a previous-value ref (`wasOpenRef`) to detect the exact

package/dist/esm/internal/use-animation-preset.js

@@ -3,7 +3,7 @@ import { ensurePresetStyles } from './ensure-preset-styles';
 /**
  * Normalizes the `animate` prop to a preset (or null) and ensures its CSS
  * is injected once into document.head. Used by PopupContent, Popover, and
- * DialogContent so they don't duplicate this logic.
+ * DialogContent so they do not duplicate this logic.
  */
 export function useAnimationPreset(animate) {
   var preset = animate != null && animate !== false ? animate : null;

package/dist/esm/internal/use-preset-styles.js

@@ -4,7 +4,7 @@ import { ensurePresetStyles } from './ensure-preset-styles';
  * Normalizes a preset prop to a value (or null) and ensures its CSS
  * is injected once into `document.head`.
  *
- * Works with any preset that has `{ css, name }` — animation presets
+ * Works with any preset that has `{ css, name }` - animation presets
  * (TAnimationPreset) satisfies this shape.
  */
 export function usePresetStyles(_ref) {

package/dist/esm/internal/use-width-from-anchor.js

@@ -4,8 +4,8 @@ import { setStyle } from './set-style';
 /**
  * Sets the width of a popover element relative to its anchor element.
  *
- * - `'match-anchor'` → `width: anchor-size(width)` — popover matches anchor width exactly.
- * - `'min-anchor'`   → `min-width: anchor-size(width)` — popover is at least as wide as anchor,
+ * - `'match-anchor'` → `width: anchor-size(width)` - popover matches anchor width exactly.
+ * - `'min-anchor'`   → `min-width: anchor-size(width)` - popover is at least as wide as anchor,
  *   but can grow wider.
  * - `'none'`         → no explicit width is set (default behaviour)
  *
@@ -30,7 +30,7 @@ export function useWidthFromAnchor(_ref) {
     var supportsAnchorSize = typeof CSS !== 'undefined' && typeof CSS.supports === 'function' && CSS.supports('width', 'anchor-size(width)');
     if (mode === 'match-anchor') {
       // Set an exact width matching the anchor.
-      // `min-inline-size` is intentionally NOT set here — it would override the
+      // `min-inline-size` is intentionally NOT set here - it would override the
       // explicit width constraint, allowing the popover to grow past the anchor's width.
       if (supportsAnchorSize) {
         return setStyle({
@@ -86,7 +86,7 @@ export function useWidthFromAnchor(_ref) {
         });
       }
 
-      // No anchor available in JS fallback — still apply min-inline-size.
+      // No anchor available in JS fallback - still apply min-inline-size.
       return setStyle({
         el: node,
         styles: [minInlineSizeStyle]

package/dist/esm/popover/create-close-event.js

@@ -2,7 +2,7 @@
  * Creates a synthetic DOM event from a `TPopoverCloseReason`.
  *
  * Useful when bridging the Popover primitive's `onClose({ reason })` callback
- * to a legacy API that expects a native DOM event — for example,
+ * to a legacy API that expects a native DOM event - for example,
  * `@atlaskit/inline-dialog`'s `onClose(event)` contract.
  *
  * - `'escape'` → `KeyboardEvent('keydown', { key: 'Escape' })`

package/dist/esm/popover/popover.js

@@ -47,7 +47,7 @@ var POPUP_ROLES = new Set(['menu', 'listbox', 'dialog', 'alertdialog', 'tree', '
  * Unopinionated top-layer primitive.
  *
  * Manages only visibility (isOpen) and animation (animate). Has no knowledge
- * of positioning — compose with `useAnchorPosition` for anchor positioning
+ * of positioning - compose with `useAnchorPosition` for anchor positioning
  * and optionally `useWidthFromAnchor` for setting the width of a popover element
  * relative to its anchor trigger.
  *
@@ -96,7 +96,7 @@ export var Popover = /*#__PURE__*/forwardRef(function Popover(_ref, ref) {
     showChildren = _useAnimatedVisibilit.showChildren,
     preset = _useAnimatedVisibilit.preset;
 
-  // ── Focus management ──
+  // Focus management
   // Initial focus: on open, moves focus into the popover (role-dependent).
   // Focus wrap: Tab/Shift+Tab cycle within the popover for dialog roles.
   // Focus restoration: handled natively by the Popover API. The browser
@@ -112,7 +112,7 @@ export var Popover = /*#__PURE__*/forwardRef(function Popover(_ref, ref) {
     role: role
   });
 
-  // ── Open layer observer registration ──
+  // Open layer observer registration
   // Notifies the open layer observer so app-coordination features
   // (closeLayers(), open-count subscriptions) work with top-layer popovers.
   //

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

@@ -89,7 +89,7 @@ export var PopupContent = /*#__PURE__*/forwardRef(function PopupContent(_ref, re
   // Note: onOpenChange composition is handled by handleOpenChange below,
   // which also adds nested-popover focus restoration.
 
-  // ── Sync aria-haspopup on the trigger with the content's role ──
+  // Sync aria-haspopup on the trigger with the content's role
   var setAriaHasPopup = ctx === null || ctx === void 0 ? void 0 : ctx.setAriaHasPopup;
   useLayoutEffect(function () {
     setAriaHasPopup === null || setAriaHasPopup === void 0 || setAriaHasPopup(roleToAriaHasPopup({
@@ -97,7 +97,7 @@ export var PopupContent = /*#__PURE__*/forwardRef(function PopupContent(_ref, re
     }));
   }, [role, setAriaHasPopup]);
 
-  // ── Focus restoration ──
+  // Focus restoration
   // The native Popover API handles focus restoration automatically for
   // popover="auto" and popover="hint":
   //   - Escape → restores focus to the previously focused element (the trigger)
@@ -124,7 +124,7 @@ export var PopupContent = /*#__PURE__*/forwardRef(function PopupContent(_ref, re
     }
     consumerOnOpenChange === null || consumerOnOpenChange === void 0 || consumerOnOpenChange(args);
 
-    // ── Nested popover focus restoration fallback ──
+    // Nested popover focus restoration fallback
     // The browser only captures `previouslyFocusedElement` for the
     // outermost popover="auto" in the stack. Inner (nested) popovers
     // have shouldRestoreFocus=false, so the browser will not restore
@@ -153,7 +153,7 @@ export var PopupContent = /*#__PURE__*/forwardRef(function PopupContent(_ref, re
     }
   }, [consumerOnOpenChange, triggerRef, role, setPopupState, willAnimate]);
 
-  // ── Anchor positioning ──
+  // Anchor positioning
   // Compose useAnchorPosition with Popover. The hook positions the popover
   // relative to the trigger element via CSS Anchor Positioning (with JS fallback).
   // Use ctx.popoverRef when inside the compound (so the trigger can also

package/dist/esm/testing/polyfill.js

@@ -102,7 +102,7 @@ function createToggleScheduler() {
       newState: newState
     });
 
-    // Task queue (setTimeout) — browser fires toggle after microtasks drain, not during
+    // Task queue (setTimeout) - browser fires toggle after microtasks drain, not during
     setTimeout(function () {
       var entry = pending.get(element);
       if (!entry) {
@@ -172,7 +172,7 @@ function createDialogToggleScheduler() {
   };
 }
 
-// ─── Popover API ────────────────────────────────────────────────────────────
+// Popover API
 // Spec: https://html.spec.whatwg.org/multipage/popover.html
 
 // `popover` attribute valid values. Empty / "auto" -> "auto"; unknown -> "manual".
@@ -535,7 +535,7 @@ if (typeof HTMLElement !== 'undefined') {
       if (isPopoverOpen({
         element: this
       })) {
-        return; // already showing — no-op
+        return; // already showing - no-op
       }
 
       // sync, cancelable on show only
@@ -715,7 +715,7 @@ if (typeof HTMLElement !== 'undefined') {
   }
 }
 
-// ─── HTMLDialogElement ──────────────────────────────────────────────────────
+// HTMLDialogElement
 // Spec: https://html.spec.whatwg.org/multipage/interactive-elements.html#the-dialog-element
 
 if (typeof HTMLDialogElement !== 'undefined') {
@@ -730,7 +730,7 @@ if (typeof HTMLDialogElement !== 'undefined') {
       }
       if (this.hasAttribute('open')) {
         if (this._isModal) {
-          return; // already modal — no-op
+          return; // already modal - no-op
         }
         throw new DOMException("Failed to execute 'showModal' on 'HTMLDialogElement': ".concat(describeElement({
           element: this
@@ -754,7 +754,7 @@ if (typeof HTMLDialogElement !== 'undefined') {
   }
   if (typeof proto.show !== 'function') {
     proto.show = function show() {
-      // Browser does not throw for disconnected elements on show — just no-op
+      // Browser does not throw for disconnected elements on show - just no-op
       if (!this.isConnected || this.hasAttribute('open')) {
         return;
       }
@@ -777,10 +777,10 @@ if (typeof HTMLDialogElement !== 'undefined') {
   if (typeof proto.close !== 'function') {
     proto.close = function close(returnValue) {
       if (!this.hasAttribute('open')) {
-        return; // already closed — no-op
+        return; // already closed - no-op
       }
 
-      // Sync beforetoggle (not cancelable) — matches browser behavior
+      // Sync beforetoggle (not cancelable) - matches browser behavior
       fireToggleEvent({
         element: this,
         type: 'beforetoggle',

package/dist/esm/testing/to-be-visible.js

@@ -5,13 +5,13 @@
  *
  * jest-dom's `toBeVisible()` does not know about the Popover API. In real
  * browsers the UA stylesheet hides `[popover]` with `display: none`, but
- * jsdom has no such stylesheet — so closed popovers incorrectly appear visible.
+ * jsdom has no such stylesheet - so closed popovers incorrectly appear visible.
  *
  * Raw jest-dom results in jsdom:
  *
- *   closed popover / child  → true  (WRONG — should be false)
+ *   closed popover / child  → true  (WRONG - should be false)
  *   open popover / child    → true  (correct)
- *   closed dialog / child   → false (correct — jsdom natively hides <dialog>)
+ *   closed dialog / child   → false (correct - jsdom natively hides <dialog>)
  *   open dialog / child     → true  (correct)
  *
  * We correct both false positives and false negatives:
@@ -62,7 +62,7 @@ expect.extend({
     var originalResult = originalToBeVisible.call(this, element);
     var topLayerState = nearestTopLayerState(element);
     if (originalResult.pass) {
-      // Original says visible — override if nearest top-layer host is closed.
+      // Original says visible - override if nearest top-layer host is closed.
       if (topLayerState === 'closed') {
         return {
           pass: false,
@@ -74,7 +74,7 @@ expect.extend({
       return originalResult;
     }
 
-    // Original says not visible — override if nearest top-layer host is open.
+    // Original says not visible - override if nearest top-layer host is open.
     // jsdom may hide [popover] elements even when they are logically open.
     if (topLayerState === 'open') {
       return {

package/dist/esm/use-arrow-navigation/is-at-current-menu-level.js

@@ -1,5 +1,23 @@
 var menuScope = '[role="menu"]';
 
+/**
+ * Recursively walks from `node` toward `container`, returning `true` when
+ * `container` is reached without crossing another `role="menu"` ancestor.
+ *
+ * Used by `isAtCurrentMenuLevel` (pattern 2) to determine if the
+ * `closestMenu` is owned by the wrapper container directly. Recursion
+ * avoids a mutable `let parent` walk and keeps the function body linear.
+ */
+function reachesContainerWithoutNestedMenu(node, container) {
+  if (!node || node === container) {
+    return node === container;
+  }
+  if (node.getAttribute('role') === 'menu') {
+    return false;
+  }
+  return reachesContainerWithoutNestedMenu(node.parentElement, container);
+}
+
 /**
  * Returns true if the element belongs to the current menu level of the
  * container, i.e. it is NOT inside a nested `role="menu"` sub-container.
@@ -66,16 +84,5 @@ export function isAtCurrentMenuLevel(element, container) {
 
   // Walk up from the closest menu to see if it is "owned" by this
   // container without passing through another role="menu" first.
-  var parent = closestMenu.parentElement;
-  while (parent && parent !== container) {
-    if (parent.getAttribute('role') === 'menu') {
-      // There is a parent menu between the closest menu and the
-      // container, so the element is in a nested sub-menu.
-      return false;
-    }
-    parent = parent.parentElement;
-  }
-
-  // The closest menu is directly inside the container (no nesting).
-  return parent === container;
+  return reachesContainerWithoutNestedMenu(closestMenu.parentElement, container);
 }

package/dist/esm/use-simple-light-dismiss/index.js

@@ -9,7 +9,7 @@ import { bind } from 'bind-event-listener';
  * **No stacking awareness:** This hook does not track a popover stack.
  * If multiple manual popovers are open and all use this hook, a single
  * Escape press or outside click will dismiss **all of them simultaneously**.
- * This is by design — the hook is intentionally simple for cases where
+ * This is by design - the hook is intentionally simple for cases where
  * only one manual popover is open at a time.
  *
  * For stacked/nested popovers that should dismiss one-at-a-time, use
@@ -46,7 +46,7 @@ export function useSimpleLightDismiss(_ref) {
     // (trigger onClick → setIsOpen(true) → React render → useEffect).
     // By the time this effect runs, the opening click has finished
     // bubbling. A capture-phase listener added now cannot retroactively
-    // fire for that event — it only fires for *future* clicks.
+    // fire for that event - it only fires for *future* clicks.
     //
     // This avoids the need for timestamp guards or deferred binding.
     // eslint-disable-next-line @atlaskit/platform/no-direct-document-usage -- binding to document for light dismiss
@@ -58,7 +58,7 @@ export function useSimpleLightDismiss(_ref) {
           return;
         }
 
-        // If the click target is inside the popover, don't dismiss.
+        // If the click target is inside the popover, do not dismiss.
         if (event.target instanceof Node && el.contains(event.target)) {
           return;
         }

package/dist/types/dialog/create-close-event.d.ts

@@ -3,7 +3,7 @@ import type { TDialogCloseReason } from './types';
  * Creates a synthetic DOM event from a `TDialogCloseReason`.
  *
  * Useful when bridging the Dialog primitive's `onClose({ reason })` callback
- * to a legacy API that expects a `KeyboardEvent` or `MouseEvent` — for example,
+ * to a legacy API that expects a `KeyboardEvent` or `MouseEvent` - for example,
  * `@atlaskit/modal-dialog`'s `onClose(event, analyticsEvent)` contract.
  *
  * - `'escape'` → `KeyboardEvent('keydown', { key: 'Escape' })`

package/dist/types/dialog/types.d.ts

@@ -49,6 +49,14 @@ type TDialogBaseProps = {
      * Additional inline styles applied to the `<dialog>` element.
      *
      * Use for custom width, height, transform, or other overrides.
+     *
+     * Rule deviation: the house rules ban unbounded `style` props on
+     * components. `Dialog` is a low-level primitive that intentionally has
+     * no visual opinions of its own; consumers such as `@atlaskit/modal-dialog`
+     * must be able to set width, height, and transforms on the native
+     * `<dialog>` element. Removing this prop would be a breaking change
+     * and is tracked separately. Keep until an xcss-based replacement
+     * exists on the primitive.
      */
     style?: CSSProperties;
     /**

package/dist/types/internal/reduced-motion.d.ts

@@ -5,7 +5,7 @@
  *
  * The result is cached after the first call via `once`. This avoids
  * calling `matchMedia` on every render. The trade-off is that if a user
- * toggles the OS reduced-motion setting mid-session, the change won't
+ * toggles the OS reduced-motion setting mid-session, the change will not
  * take effect until the next page load. This is acceptable because
  * toggling this system-level preference mid-session is extremely rare,
  * and a page refresh is a reasonable expectation for system accessibility

package/dist/types/internal/resolve-placement.d.ts

@@ -30,8 +30,8 @@ export type TPlacement = {
  * CSS length string (e.g. `token('space.100')`).
  *
  * The two offset axes are named for clarity at the call site:
- *   - `gap`     — distance away from the trigger along the placement axis
- *   - `crossAxisShift`  — nudge along the trigger edge (perpendicular to the placement axis)
+ *   - `gap`     - distance away from the trigger along the placement axis
+ *   - `crossAxisShift`  - nudge along the trigger edge (perpendicular to the placement axis)
  */
 export type TPlacementOptions = {
     axis?: 'block' | 'inline';