@jsenv/dom 0.17.0 → 0.17.1

package/dist/jsenv_dom.js

@@ -4592,13 +4592,6 @@ const DEFAULT_BEHAVIORS = [
     },
     // no fallback: only claims Tab, other keys continue to next entries
   },
-  {
-    // Escape natively dismisses only <dialog> elements
-    test: (el) => el.tagName === "DIALOG" || Boolean(el.closest("dialog")),
-    keys: {
-      escape: "dismiss",
-    },
-  },
   {
     test: (el) => el.matches("input[type='radio'], input[type='checkbox']"),
     keys: {
@@ -4748,6 +4741,18 @@ const DEFAULT_BEHAVIORS = [
       enter: "activate",
     },
   },
+  {
+    // Escape natively dismisses only <dialog> elements. Deliberately late in
+    // the list: the focused element gets first claim on Escape, because the
+    // browser resolves the close request innermost-first. A non-empty
+    // <input type="search"> inside a dialog consumes the first Escape to clear
+    // itself and only a second one reaches the dialog — reporting "dismiss"
+    // here would let our own Escape-to-close shortcuts fire on the first press.
+    test: (el) => el.tagName === "DIALOG" || Boolean(el.closest("dialog")),
+    keys: {
+      escape: "dismiss",
+    },
+  },
   {
     // Non-interactive elements: browser scrolls on Space and arrow keys
     test: () => true,
@@ -11568,6 +11573,10 @@ const MIN_CONTENT_VISIBILITY_RATIO = 0.6;
  *
  * A bit like https://tetherjs.dev/ but different
  */
+// The event type observeSize() reports with — recognized by check() as "the
+// change is in another element, not in the tracked rect".
+const OBSERVED_ELEMENT_SIZE_CHANGE = "observed_element_size_change";
+
 const visibleRectEffect = (
   element,
   update,
@@ -11810,6 +11819,18 @@ const visibleRectEffect = (
       });
     };
 
+    // An observeSize() delivery reports a size change in some *other*
+    // element — this one's own rect and the viewport are both typically
+    // untouched by it, so the dedup below would skip every single one,
+    // defeating the whole point of observeSize (a popover reconsidering its
+    // placement once its own content shrinks/grows, a callout re-measuring
+    // against its message body).
+    if (event.type === OBSERVED_ELEMENT_SIZE_CHANGE) {
+      lastVisibleRect = visibleRect;
+      lastViewportRect = viewportRect;
+      notify();
+      return;
+    }
     const visibleRectChanged =
       !lastVisibleRect ||
       lastVisibleRect.left !== visibleRect.left ||
@@ -12214,7 +12235,7 @@ const visibleRectEffect = (
       pendingFrame = requestAnimationFrame(() => {
         pendingFrame = null;
         check(
-          new CustomEvent("observed_element_size_change", {
+          new CustomEvent(OBSERVED_ELEMENT_SIZE_CHANGE, {
             detail: { width, height },
           }),
         );

package/package.json

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