@react-aria/overlays 3.12.1 → 3.13.0

package/dist/module.js

@@ -339,7 +339,7 @@ function $2a41e45df1593e64$export$d39e1813b3bdd0e1(props) {
     ];
     let updatePosition = (0, $k7QOs$useCallback)(()=>{
         if (shouldUpdatePosition === false || !isOpen || !overlayRef.current || !targetRef.current || !scrollRef.current || !boundaryElement) return;
-        setPosition((0, $edcf132a9284368a$export$b3ceb0cbf1056d98)({
+        let position = (0, $edcf132a9284368a$export$b3ceb0cbf1056d98)({
             placement: $2a41e45df1593e64$var$translateRTL(placement, direction),
             overlayNode: overlayRef.current,
             targetNode: targetRef.current,
@@ -350,7 +350,13 @@ function $2a41e45df1593e64$export$d39e1813b3bdd0e1(props) {
             offset: offset,
             crossOffset: crossOffset,
             maxHeight: maxHeight
-        }));
+        });
+        // Modify overlay styles directly so positioning happens immediately without the need of a second render
+        // This is so we don't have to delay autoFocus scrolling or delay applying preventScroll for popovers
+        Object.keys(position.position).forEach((key)=>overlayRef.current.style[key] = position.position[key] + "px");
+        overlayRef.current.style.maxHeight = position.maxHeight != null ? position.maxHeight + "px" : undefined;
+        // Trigger a set state for a second render anyway for arrow positioning
+        setPosition(position);
     // eslint-disable-next-line react-hooks/exhaustive-deps
     }, deps);
     // Update position when anything changes
@@ -1152,20 +1158,35 @@ let $5e3802645cc19319$var$observerStack = [];
 function $5e3802645cc19319$export$1c3ebcada18427bf(targets, root = document.body) {
     let visibleNodes = new Set(targets);
     let hiddenNodes = new Set();
-    let walker = document.createTreeWalker(root, NodeFilter.SHOW_ELEMENT, {
-        acceptNode (node) {
-            // If this node is a live announcer, add it to the set of nodes to keep visible.
-            if ((node instanceof HTMLElement || node instanceof SVGElement) && node.dataset.liveAnnouncer === "true") visibleNodes.add(node);
+    let walk = (root)=>{
+        // Keep live announcer and top layer elements (e.g. toasts) visible.
+        for (let element of root.querySelectorAll("[data-live-announcer], [data-react-aria-top-layer]"))visibleNodes.add(element);
+        let acceptNode = (node)=>{
             // Skip this node and its children if it is one of the target nodes, or a live announcer.
             // Also skip children of already hidden nodes, as aria-hidden is recursive. An exception is
             // made for elements with role="row" since VoiceOver on iOS has issues hiding elements with role="row".
             // For that case we want to hide the cells inside as well (https://bugs.webkit.org/show_bug.cgi?id=222623).
             if (visibleNodes.has(node) || hiddenNodes.has(node.parentElement) && node.parentElement.getAttribute("role") !== "row") return NodeFilter.FILTER_REJECT;
             // Skip this node but continue to children if one of the targets is inside the node.
-            if (targets.some((target)=>node.contains(target))) return NodeFilter.FILTER_SKIP;
+            for (let target of visibleNodes){
+                if (node.contains(target)) return NodeFilter.FILTER_SKIP;
+            }
             return NodeFilter.FILTER_ACCEPT;
+        };
+        let walker = document.createTreeWalker(root, NodeFilter.SHOW_ELEMENT, {
+            acceptNode: acceptNode
+        });
+        // TreeWalker does not include the root.
+        let acceptRoot = acceptNode(root);
+        if (acceptRoot === NodeFilter.FILTER_ACCEPT) hide(root);
+        if (acceptRoot !== NodeFilter.FILTER_REJECT) {
+            let node = walker.nextNode();
+            while(node != null){
+                hide(node);
+                node = walker.nextNode();
+            }
         }
-    });
+    };
     let hide = (node)=>{
         var _refCountMap_get;
         let refCount = (_refCountMap_get = $5e3802645cc19319$var$refCountMap.get(node)) !== null && _refCountMap_get !== void 0 ? _refCountMap_get : 0;
@@ -1179,11 +1200,7 @@ function $5e3802645cc19319$export$1c3ebcada18427bf(targets, root = document.body
     // If there is already a MutationObserver listening from a previous call,
     // disconnect it so the new on takes over.
     if ($5e3802645cc19319$var$observerStack.length) $5e3802645cc19319$var$observerStack[$5e3802645cc19319$var$observerStack.length - 1].disconnect();
-    let node = walker.nextNode();
-    while(node != null){
-        hide(node);
-        node = walker.nextNode();
-    }
+    walk(root);
     let observer = new MutationObserver((changes)=>{
         for (let change of changes){
             if (change.type !== "childList" || change.addedNodes.length === 0) continue;
@@ -1192,9 +1209,15 @@ function $5e3802645cc19319$export$1c3ebcada18427bf(targets, root = document.body
             if (![
                 ...visibleNodes,
                 ...hiddenNodes
-            ].some((node)=>node.contains(change.target))) for (let node of change.addedNodes){
-                if ((node instanceof HTMLElement || node instanceof SVGElement) && node.dataset.liveAnnouncer === "true") visibleNodes.add(node);
-                else if (node instanceof Element) hide(node);
+            ].some((node)=>node.contains(change.target))) {
+                for (let node of change.removedNodes)if (node instanceof Element) {
+                    visibleNodes.delete(node);
+                    hiddenNodes.delete(node);
+                }
+                for (let node1 of change.addedNodes){
+                    if ((node1 instanceof HTMLElement || node1 instanceof SVGElement) && (node1.dataset.liveAnnouncer === "true" || node1.dataset.reactAriaTopLayer === "true")) visibleNodes.add(node1);
+                    else if (node1 instanceof Element) walk(node1);
+                }
             }
         }
     });
@@ -1247,7 +1270,6 @@ function $5e3802645cc19319$export$1c3ebcada18427bf(targets, root = document.body
 
 
 
-
 function $f2f8a6077418541e$export$542a6fd13ac93354(props, state) {
     let { triggerRef: triggerRef , popoverRef: popoverRef , isNonModal: isNonModal , isKeyboardDismissDisabled: isKeyboardDismissDisabled , ...otherProps } = props;
     let { overlayProps: overlayProps , underlayProps: underlayProps  } = (0, $a11501f3d1d39e6c$export$ea8f71083e90600f)({
@@ -1264,18 +1286,8 @@ function $f2f8a6077418541e$export$542a6fd13ac93354(props, state) {
         isOpen: state.isOpen,
         onClose: null
     });
-    // Delay preventing scroll until popover is positioned to avoid extra scroll padding.
-    // This requires a layout effect so that positioning has been committed to the DOM
-    // by the time usePreventScroll measures the element.
-    let [isPositioned, setPositioned] = (0, $k7QOs$useState)(false);
-    (0, $k7QOs$useLayoutEffect)(()=>{
-        if (!isNonModal && placement) setPositioned(true);
-    }, [
-        isNonModal,
-        placement
-    ]);
     (0, $49c51c25361d4cd2$export$ee0f7cc6afcd1c18)({
-        isDisabled: isNonModal || !isPositioned
+        isDisabled: isNonModal
     });
     (0, $k7QOs$useLayoutEffect)(()=>{
         if (state.isOpen && !isNonModal && popoverRef.current) return (0, $5e3802645cc19319$export$1c3ebcada18427bf)([