@jsenv/dom 0.17.84 → 0.17.86

package/dist/jsenv_dom.js

@@ -4976,14 +4976,19 @@ const getFocusVisibilityInfo = (node, { excludeAriaHidden } = {}) => {
       }
       // Continue checking ancestors
     }
-    if (elementIsDialog(nodeOrAncestor) && !nodeOrAncestor.open) {
-      return { visible: false, reason: "inside closed dialog element" };
-    }
+    const isPopover =
+      nodeOrAncestor.popover !== null && nodeOrAncestor.popover !== undefined;
+    const isOpenPopover = isPopover && nodeOrAncestor.matches(":popover-open");
+    // A dialog shown as a popover (showPopover()) is open without `open`,
+    // which only show() and showModal() set.
     if (
-      nodeOrAncestor.popover !== null &&
-      nodeOrAncestor.popover !== undefined &&
-      !nodeOrAncestor.matches(":popover-open")
+      elementIsDialog(nodeOrAncestor) &&
+      !nodeOrAncestor.open &&
+      !isOpenPopover
     ) {
+      return { visible: false, reason: "inside closed dialog element" };
+    }
+    if (isPopover && !isOpenPopover) {
       return { visible: false, reason: "inside closed popover element" };
     }
     // Open popovers and open dialogs render in the top layer: they escape
@@ -4992,11 +4997,7 @@ const getFocusVisibilityInfo = (node, { excludeAriaHidden } = {}) => {
     if (elementIsDialog(nodeOrAncestor) && nodeOrAncestor.open) {
       break;
     }
-    if (
-      nodeOrAncestor.popover !== null &&
-      nodeOrAncestor.popover !== undefined &&
-      nodeOrAncestor.matches(":popover-open")
-    ) {
+    if (isOpenPopover) {
       break;
     }
     nodeOrAncestor = nodeOrAncestor.parentNode;
@@ -5251,8 +5252,8 @@ const findFocusable = (element, { exclude } = {}) => {
 // - `clip` is not one. It clips and stops there: it has no scroll box at all,
 //   and scrollTop/scrollLeft on it read back 0 whatever they are set to. That
 //   is the whole reason a layout reaches for it (see navi's
-//   docs/mobile_layout_pitfalls.md), so it is never included, whatever the
-//   caller asks for.
+//   docs/safe_area.md#a-document-wider-than-the-screen), so it is never
+//   included, whatever the caller asks for.
 // `visible` is the other non-scrolling value, and the only one on which the
 // content still reaches the ancestors.
 const isScrollable = (element, { includeHidden } = {}) => {
@@ -15039,10 +15040,11 @@ const startDragTo = (
  * A press that WOULD be a drag, and is not.
  *
  * Recognized exactly as `startDragTo` recognizes it: the intent is established by
- * the same threshold — a mouse travelling, a finger holding still, the first
- * pixel inside a `[data-drag-on-contact]`. What differs is what happens once it
- * is established: nothing is grabbed, nothing translates, and `onRefuse` is told
- * at the instant the grab would have been acquired.
+ * the same threshold — a mouse travelling a few pixels, a finger holding still,
+ * or travelling those same few pixels inside a `[data-drag-on-contact]`. What
+ * differs is what happens once it is established: nothing is grabbed, nothing
+ * translates, and `onRefuse` is told at the instant the grab would have been
+ * acquired.
  *
  * That instant is the whole point. An object that stays put under the hand and
  * says nothing reads as a screen that is broken, and the hand pulls harder; the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/dom",
-  "version": "0.17.84",
+  "version": "0.17.86",
   "type": "module",
   "description": "DOM utilities for writing frontend code",
   "repository": {